[PATCH] D73317: [GISelKnownBits] Add support for PHIs

Quentin Colombet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 27 09:51:19 PST 2020


qcolombet added a comment.

> and having the builder silently ignore the explicitly setInsertPt. I think that behavior should be opt-in only to CSE, and obvious when it's being done.

Fair, but do we really need to build something if a copy is what is going to be inserted?
I am assuming the answer would be a case by case answer.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D73317





More information about the llvm-commits mailing list