Closed
Bug 778301
Opened 12 years ago
Closed 12 years ago
Remove old change for llvm.org/pr11003
Categories
(Core :: JavaScript Engine, defect)
Tracking
()
RESOLVED
FIXED
mozilla17
People
(Reporter: espindola, Assigned: espindola)
Details
Attachments
(1 file)
1.02 KB,
patch
|
luke
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Comment 1•12 years ago
|
||
Comment on attachment 646717 [details] [diff] [review] remove it I am testing and benchmarking it, but I think review can go in parallel.
Attachment #646717 -
Flags: review?(luke)
Assignee | ||
Updated•12 years ago
|
Assignee: general → respindola
Status: NEW → ASSIGNED
Comment 2•12 years ago
|
||
Which clang version was this fixed in? I have another piece of code in Gecko which used to hit an LLVM bug which has since been fixed. I wonder what our policy for keeping the code buildable with older clang versions should be...
![]() |
||
Updated•12 years ago
|
Attachment #646717 -
Attachment is patch: true
Attachment #646717 -
Flags: review?(luke) → review+
Assignee | ||
Comment 3•12 years ago
|
||
It was fixed in r141137, from Oct 4, 2011. The fix in included clang 3.0 and 3.1. It is also in apple's xcode 4.4 (not sure about older versions). I think on OS X we can build with clang >= 3.0, but I am not sure.
Comment 5•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/0dbf799a2371
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla17
You need to log in
before you can comment on or make changes to this bug.
Description
•