[PATCH] D57568: [SCEV] Don't bother preserving LCSSA in SCEV

Sanjoy Das via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 1 21:07:01 PST 2019


sanjoy accepted this revision.
sanjoy added a comment.

I'm okay with the direction, but I'm also worried that this breaks some subtle invariant between SCEV and SCEVExpander.  Have you tried bootstrapping clang with this change to see if that shakes out any issues?


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

https://reviews.llvm.org/D57568





More information about the llvm-commits mailing list