[PATCH] D69369: [NFC] Add a unit test exposing lack of SCEV invalidation in LICM during code hoisting

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 28 10:42:53 PDT 2019


reames requested changes to this revision.
reames added a comment.
This revision now requires changes to proceed.

I think you can write this test better as an lit test with strict validation.  I assume SCEV has an expensive asserts mode - if not, we should add one - and thus this can be written as a lit test with an extra flag to opt.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D69369/new/

https://reviews.llvm.org/D69369





More information about the llvm-commits mailing list