[PATCH] D71249: [LV] Set name for vector preheader and trip count check blocks

Renato Golin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 11 02:25:50 PST 2019


rengolin added a comment.

This change may conflict with your other two. How are we supposed to review them? Are they a set?

If so, what's the order in which they have to be applied?

If this is just a naming issue, and the test changes look all related to naming, then I agree with @lebedev.ri.

This should be a suffix to the current name, otherwise you'll end up with multiple %tc.check.N still having to find which branch leads where.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71249





More information about the llvm-commits mailing list