Closed
Bug 1466155
Opened 7 years ago
Closed 7 years ago
Remove unused labels in CacheIRCompiler::emitGuardIsInt32
Categories
(Core :: JavaScript Engine: JIT, enhancement, P3)
Core
JavaScript Engine: JIT
Tracking
()
RESOLVED
FIXED
mozilla66
| Tracking | Status | |
|---|---|---|
| firefox66 | --- | fixed |
People
(Reporter: mgaudet, Assigned: championshuttler, Mentored)
Details
(Keywords: good-first-bug)
Attachments
(1 file)
CacheIRCompiler::emitIsInt32 has two unused label variables:
https://searchfox.org/mozilla-central/rev/38bcf897f1fa19c1eba441a611cf309482e0d6e5/js/src/jit/CacheIRCompiler.cpp#1376
They can safely be removed.
Comment 1•7 years ago
|
||
Hey Matthew,
I would like work on this bug. I understand that the line pointed out by you needs to be removed, I am currently building the source, I will upload the patch once I have built Firefox.
Regards,
Abhishek [:avp]
Updated•7 years ago
|
Assignee: nobody → abhishekp.bugzilla
Updated•7 years ago
|
Status: NEW → ASSIGNED
| Assignee | ||
Comment 2•7 years ago
|
||
Hi @Abhishek are you working on it? If not can i work on it?
Thanks
Flags: needinfo?(abhishekp.bugzilla)
| Assignee | ||
Comment 3•7 years ago
|
||
Updated•7 years ago
|
Assignee: abhishekp.bugzilla → shivams2799
Pushed by tcampbell@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/d70fba3550b4
Remove unused labels in CacheIRCompiler::emitGuardIsInt32. r=tcampbell
Comment 5•7 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox66:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla66
Updated•7 years ago
|
status-firefox62:
affected → ---
Updated•7 years ago
|
Flags: needinfo?(abhishekp.bugzilla)
You need to log in
before you can comment on or make changes to this bug.
Description
•