| 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 Bugs | Assignee: | Andy Estes <aestes> | ||||
| Status: | RESOLVED FIXED | ||||||
| Severity: | Normal | ||||||
| Priority: | P2 | ||||||
| Version: | 528+ (Nightly build) | ||||||
| Hardware: | Unspecified | ||||||
| OS: | Unspecified | ||||||
| Attachments: |
|
||||||
|
Description
Andy Estes
2013-06-28 16:12:12 PDT
Created attachment 205752 [details]
Patch
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/? 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.
Committed r152195: <http://trac.webkit.org/changeset/152195> |