[PATCH] D54899: [LoopStrengthReduce] ComplexityLimit as an option

Gil Rapaport via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 27 05:37:53 PST 2018


gilr added inline comments.


================
Comment at: test/Transforms/LoopStrengthReduce/ARM/complexity.ll:28
+  %cmp654 = icmp eq i32 %filter_dim, 0
+  %0 = shl i32 %out_width, 2
+  %1 = add i32 %filter_dim, -1
----------------
Could you get rid of the %[0-9]+ implicit variable names? ('opt -instnamer' does that for you)
Those are painful when manually editing tests.


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

https://reviews.llvm.org/D54899





More information about the llvm-commits mailing list