[PATCH] D66334: [LoopUnroll] Handle certain PHIs in full unrolling properly.

Bevin Hansson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 16 04:14:29 PDT 2019


ebevhan marked an inline comment as done.
ebevhan added inline comments.


================
Comment at: test/Transforms/LoopUnroll/full-unroll-miscompile.ll:8
+; CHECK-LABEL: main
+; CHECK: %.lcssa10.lcssa = phi i16 [ %tmp2, %for.cond.cleanup3 ]
+
----------------
lebedev.ri wrote:
> passing-by remark: it's way too fragile to test exact value names,
> you may want to test more of the output (capturing the value names elsewhere),
> if not all of it.
I suppose I can generate the checks with the python script.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D66334





More information about the llvm-commits mailing list