[PATCH] D32180: [InstSimplify] Deduce correct type for vector gep

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 19 06:52:17 PDT 2017


spatel added a comment.

Since we already have a test file specifically for vector gep simplifications (test/Transforms/InstSimplify/vector_gep.ll), can you please add these new tests there? I find it easier to locate and fix bugs when related tests are co-located. A comment/link back to http://bugs.llvm.org/show_bug.cgi?id=32697 above the regression test also saves time for future bug hunting. Thanks!


https://reviews.llvm.org/D32180





More information about the llvm-commits mailing list