[PATCH] D42290: [SCEV] Clear poison flags during expansion of SCEV

Max Kazantsev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Feb 25 20:40:04 PST 2018


mkazantsev added a comment.

Hi @sanjoy @wmi @hfinkel, is there any discussion ongoing about this patch? There is more than month without progress, should we maybe start a discussion here?

I'm personaly also inclined to think that this caching logic is wrong and we should get rid of it, but will clearly have consequences from compile time point, so it's definitely worth discussing.


https://reviews.llvm.org/D42290





More information about the llvm-commits mailing list