[PATCH] D38856: [IPSCCP] Remove calls without side effects

Davide Italiano via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 11 13:32:32 PST 2017


davide accepted this revision.
davide added a comment.
This revision is now accepted and ready to land.

I don't think this function is general enough to live in `Instruction.h`, I'd keep it private to SCCP for now.
I think we need to reach consensus on how this will look like in the end, but this is probably fine for now.
I'd give @majnemer / @dberlin another change of commenting, but I guess this is fine to get in.


https://reviews.llvm.org/D38856





More information about the llvm-commits mailing list