Closed Bug 1575219 Opened 4 months ago Closed 3 months ago

Remove unused FailurePath from BaselineCacheIRCompiler::emitCompareStringResult

Categories

(Core :: JavaScript Engine: JIT, task, P3)

task

Tracking

()

RESOLVED FIXED
mozilla71
Tracking Status
firefox71 --- fixed

People

(Reporter: anba, Assigned: a.munoz3327, Mentored)

Details

(Keywords: good-first-bug)

Attachments

(2 files)

BaselineCacheIRCompiler::emitCompareStringResult creates a FailurePath, but never uses it.

Matthew, do you want to mentor this issue as a good-first-bug?

Flags: needinfo?(mgaudet)
Priority: -- → P3

Can Do!

Mentor: mgaudet
Flags: needinfo?(mgaudet)
Keywords: good-first-bug

Hi Matthew, here's a patch for this. This is my first open-source contribution so let me know if I missed or misunderstood something.

Hey Ayrton;

This looks like it's definitely on the right path. There's a couple steps yet.

You'll want to follow the walkthrough here, and links from there on setting up phabricator and mozilla's review tool moz-phab.

Before you submit the patch, your commit messages should look something like this

Bug 1575219 - Remove unused failure path from BaselineCacheIRCompiler::emitCompareStringResult r?mgaudet

That way, the patch will get linked to this bug as well as directed to me for review.

After you get that up, what'll happen is I'll download your patch, apply it and push it to the mozilla try server which is how we interact with our pre-delivery continuous integration testing. Then, once that's done, I'll approve your patch, and submit it for landing, assuming everything goes smoothly.

Then after that we'll have to find you something new to work on!

Pushed by mgaudet@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/64109c67702a
Remove unused failure path from BaselineCacheIRCompiler::emitCompareStringResult r=mgaudet
Status: NEW → RESOLVED
Closed: 3 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla71
Assignee: nobody → a.munoz3327
You need to log in before you can comment on or make changes to this bug.