Page MenuHomePhabricator

PR34603 Fix by Early-CSE extension

Authored by evstupac on Oct 31 2017, 11:56 AM.



This is an alternative patch that solves PR34603:

Even if it is fixed by the patch Early-CSE extension could be useful for some cases.

Diff Detail


Event Timeline

evstupac created this revision.Oct 31 2017, 11:56 AM
dberlin edited edge metadata.Oct 31 2017, 12:28 PM

I'll repeat what i said on the bug
FWIW: Given the choice, i would much rather fix the underlying problem then patch various passes to work around some small subset of it (here, load of selected addresses)
This is underway right now, actually.

I would be more amenable to the latter approach if there was data that we could get, say, 90% of the gain of changing the canonical form by doing so.

I have doubts this is the case, but if you've got data that says it is, ...

evstupac abandoned this revision.May 2 2018, 12:08 PM