[PATCH] D26428: Update vectorization debug info unittest.

Paul Robinson via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 9 10:58:23 PST 2016


probinson added a comment.

I built opt r286158 and r286159.  Running the modified test through both copies of opt, I get identical output .ll files.  This tells me that the modified test does not actually test r286159.  I should get different outputs, that reflect the code change.



================
Comment at: test/Transforms/LoopVectorize/debugloc.ll:8
 ; CHECK: for.body.lr.ph
 ; CHECK:   cmp.zero = icmp eq i64 {{.*}}, 0, !dbg ![[LOC:[0-9]+]]
 ; CHECK: vector.body
----------------
This line sets the LOC variable but it is no longer used elsewhere.  


https://reviews.llvm.org/D26428





More information about the llvm-commits mailing list