[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`.


More information about the llvm-commits mailing list