[PATCH] D31678: [InstCombine] Fix change flag handling to report all IR changes up to the pass manager or preseved analyses

Chandler Carruth via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 10 14:23:10 PDT 2017


chandlerc added a comment.

In https://reviews.llvm.org/D31678#722169, @craig.topper wrote:

> I can certainly try. What exactly were you envisioning?


Alternatively or perhaps additionally, can you contrive a test case where we failed to invalidate an analysis but mutated the IR prior to this change? That seems like a good edge case test for the thing that would actually go wrong.


https://reviews.llvm.org/D31678





More information about the llvm-commits mailing list