[PATCH] D18266: [IndVars] Fix PR26974: make sure replaceCongruentIVs doesn't break LCSSA

Sanjoy Das via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 18 11:07:47 PDT 2016


sanjoy added inline comments.

================
Comment at: test/Transforms/IndVarSimplify/rewrite-iv-lcssa.ll:3
@@ +2,3 @@
+
+; RUN: opt -indvars  -S < %s | FileCheck %s
+
----------------
Normally, I see tests put the `; RUN` line at the very top.

Also, since this is fixing a PR, the convention I've generally observed is to call the test file `prXXXX.ll`.


http://reviews.llvm.org/D18266





More information about the llvm-commits mailing list