Closed Bug 1377566 Opened 8 years ago Closed 8 years ago

IonBuilder::ensureDefiniteTypeSet is not used

Categories

(Core :: JavaScript Engine: JIT, enhancement)

Product:
Core
Component:
JavaScript Engine: JIT
Type:
enhancement
Priority:
Not set
Severity:
trivial

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla57
Tracking Flags:
Tracking Status
firefox57 --- fixed

People

(Reporter: arai, Assigned: arai)

Details

Attachments

(1 file)

Remove unused IonBuilder::ensureDefiniteTypeSet.
3.03 KB, patch
jandem
: review+
Details | Diff | Splinter Review
there's no callsite for IonBuilder::ensureDefiniteTypeSet. we can just remove it or use it where appropriate.
I might use it in bug 1366263.
See Also: → 1366263
I didn't use it in bug 1366263.
See Also: 1366263 →
so, this is added in bug 953164, and was used in IonBuilder::filterTypesAtTest that no more exist. I'll remove this
Assignee: nobody → arai.unmht
Status: NEW → ASSIGNED
Just removed unused definition/declaration.
Attachment #8894801 - Flags: review?(jdemooij)
Comment on attachment 8894801 [details] [diff] [review] Remove unused IonBuilder::ensureDefiniteTypeSet. Review of attachment 8894801 [details] [diff] [review]: ----------------------------------------------------------------- Thanks!
Attachment #8894801 - Flags: review?(jdemooij) → review+
Pushed by arai_a@mac.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/9839ea550229 Remove unused IonBuilder::ensureDefiniteTypeSet. r=jandem
https://hg.mozilla.org/mozilla-central/rev/9839ea550229
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-firefox57: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: