[PATCH] D33195: [Polly] Add invalidation of the loop SCEVs after merge block generation

Michael Kruse via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 15 08:19:42 PDT 2017


Meinersbur added a comment.

Hi Baranidharan,

thank you for the patch. Could you please add a test case as well (e.g. the .ll file from llvm.org/PR32536, minimized as far as possible and a `CHECK:`-line for something that is relevant. e.g. that the `%num.0 = phi i32 [ %add, %for.body15 ], [ 0, %for.cond13.pre_entry_bb ]` is in the right position)



================
Comment at: lib/CodeGen/BlockGenerators.cpp:784
       llvm_unreachable("Unexpected statement type found");
+  // Invalidate SCEV of loops surrounding EscapeUsers
+  for (const auto &EscapeMapping : EscapeMap) {
----------------
Please add a newline before.


https://reviews.llvm.org/D33195





More information about the llvm-commits mailing list