This is an archive of the discontinued LLVM Phabricator instance.

[InstCombine] Pass a proper context instruction to all of the calls into InstSimplify
ClosedPublic

Authored by craig.topper on May 26 2017, 1:37 PM.

Diff Detail

Repository
rL LLVM

Event Timeline

craig.topper created this revision.May 26 2017, 1:37 PM
spatel edited edge metadata.Jun 2 2017, 9:32 AM

This matches my understanding of the follow-up discussion in D33567 :
http://lists.llvm.org/pipermail/llvm-commits/Week-of-Mon-20170522/456940.html
...but I defer to @hfinkel @dberlin for approval in case I got that wrong.

@hfinkel @dberlin does this patch look ok to you?

dberlin accepted this revision.Jun 8 2017, 5:25 PM

This seems right.
I

This revision is now accepted and ready to land.Jun 8 2017, 5:25 PM
This revision was automatically updated to reflect the committed changes.