Closed Bug 943442 Opened 8 years ago Closed 8 years ago
Reading the code in MIRGraph::allocDefinitionId I was curious whether incrementing the counter by 2 was significant. It seems not to be the case. Attached is a patch which changes to increment by 1, so that it looks less interesting, and potentially to make it useful for indexing into side tables.
Attachment #8338594 - Flags: review?(dvander)
And a related follow-up patch, MDefinitions can be compared by pointer, since two MDefinitions never share the same id() value.
Attachment #8338597 - Flags: review?(dvander)
Attachment #8338594 - Flags: review?(dvander) → review+
Attachment #8338597 - Flags: review?(dvander) → review+
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla29
You need to log in before you can comment on or make changes to this bug.