Closed
Bug 1007107
Opened 10 years ago
Closed 10 years ago
Function branchTestMagicValue is wrong on X64, ARM and MIPS
Categories
(Core :: JavaScript Engine: JIT, defect)
Tracking
()
RESOLVED
FIXED
mozilla32
People
(Reporter: rankov, Assigned: rankov)
Details
Attachments
(2 files, 2 obsolete files)
4.07 KB,
patch
|
rankov
:
review+
|
Details | Diff | Splinter Review |
1.74 KB,
patch
|
nbp
:
review+
|
Details | Diff | Splinter Review |
Function branchTestMagicValue in MacroAssembler has a bug in the case when condition is NotEqual.
Assignee | ||
Comment 1•10 years ago
|
||
Carry review from previous patch. This patch has been moved here from bug 994716 after review.
Attachment #8418720 -
Flags: review+
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → branislav.rankov
Assignee | ||
Comment 2•10 years ago
|
||
Assignee | ||
Comment 3•10 years ago
|
||
Comment on attachment 8425496 [details] [diff] [review] test-magic-value-mips-arm-x64.patch I changed this to be the same for ARM, X64 and MIPS. If you think that this is better than previous patch, I will make the previous patch obsolete.
Attachment #8425496 -
Flags: review?(nicolas.b.pierron)
Comment 4•10 years ago
|
||
Comment on attachment 8425496 [details] [diff] [review] test-magic-value-mips-arm-x64.patch Review of attachment 8425496 [details] [diff] [review]: ----------------------------------------------------------------- Nice! :) Do the same thing for x86. I have only one doubt on x64, where I guess we might store the Value as a relocatable value. I guess it might not be that important, but it would be better to check about that on try.
Attachment #8425496 -
Flags: review?(nicolas.b.pierron) → review+
Assignee | ||
Comment 5•10 years ago
|
||
Carry review from previous patch. Updated r=nbp.
Attachment #8418720 -
Attachment is obsolete: true
Attachment #8425496 -
Attachment is obsolete: true
Attachment #8426935 -
Flags: review+
Assignee | ||
Comment 6•10 years ago
|
||
Attachment #8426936 -
Flags: review?(nicolas.b.pierron)
Assignee | ||
Comment 7•10 years ago
|
||
(In reply to Nicolas B. Pierron [:nbp] from comment #4) > Nice! :) > Do the same thing for x86. Thanks. I just made a patch for this too. > I have only one doubt on x64, where I guess we might store the Value as a > relocatable value. I guess it might not be that important, but it would be > better to check about that on try. I have put both patches to try: https://tbpl.mozilla.org/?tree=Try&rev=5b3a33d3f9fe
Updated•10 years ago
|
Attachment #8426936 -
Flags: review?(nicolas.b.pierron) → review+
Assignee | ||
Comment 8•10 years ago
|
||
Checkin is needed. Here is try link: https://tbpl.mozilla.org/?tree=Try&rev=5b3a33d3f9fe
Keywords: checkin-needed
Comment 9•10 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/4e45c6143959 https://hg.mozilla.org/integration/mozilla-inbound/rev/cc1bff97f035
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/4e45c6143959 https://hg.mozilla.org/mozilla-central/rev/cc1bff97f035
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla32
You need to log in
before you can comment on or make changes to this bug.
Description
•