Bug 118208

Summary: [Mac] Newer versions of clang use a default template instantiation depth that is too small for TestWebKitAPI
Product: WebKit Reporter: Andy Estes <aestes>
Component: New BugsAssignee: Andy Estes <aestes>
Status: RESOLVED FIXED    
Severity: Normal    
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch mrowe: review+

Andy Estes
Reported 2013-06-28 16:12:12 PDT
[Mac] Newer versions of clang use a default template instantiation depth that is too small for TestWebKitAPI
Attachments
Patch (1.72 KB, patch)
2013-06-28 16:17 PDT, Andy Estes 		mrowe: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Andy Estes
Comment 1 2013-06-28 16:17:22 PDT
Created attachment 205752 [details] Patch
Tim Horton
Comment 2 2013-06-28 16:18:57 PDT
Comment on attachment 205752 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=205752&action=review > Tools/ChangeLog:8 > + Newer versions of clang use a default template depth to 128, but this s/to/of/?
Mark Rowe (bdash)
Comment 3 2013-06-28 16:24:08 PDT
Comment on attachment 205752 [details] Patch What Tim said. It may be worth briefly summarizing why it's ok for us to bump the depth back up when clang has changed it downwards, or even just putting a reference in to the radar you mentioned to me. Something to give us a reference point if we eventually wonder why we added this flag.
Andy Estes
Comment 4 2013-06-28 16:40:52 PDT
Committed r152195: <http://trac.webkit.org/changeset/152195>
Note You need to log in before you can comment on or make changes to this bug.