Page MenuHomePhabricator

PR34603 Fix by Early-CSE extension
AbandonedPublic

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

Details

Summary

This is an alternative patch that solves PR34603:
https://bugs.llvm.org/show_bug.cgi?id=34603

Even if it is fixed by https://reviews.llvm.org/D38566 the patch Early-CSE extension could be useful for some cases.

Diff Detail

Repository
rL LLVM

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