[PATCH] D69686: [LVI][CVP] Use block value in getPredicateAt()

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 1 09:58:59 PDT 2019


nikic added a comment.

@lebedev.ri I added a couple of comments on the JT tests, probably after you opened the tab ^^ The reason for the diffs is that we now evaluate the condition to a constant, while previously these were phi threaded. It has a bit of an odd effect here, but I would expect that in practice folding a branch completely is generally preferable.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69686





More information about the llvm-commits mailing list