[PATCH] D57053: Extra processing for BitCast + PHI in InstCombine

Guozhi Wei via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 6 14:53:35 PST 2019


Carrot added a comment.
Herald added a project: LLVM.

I couldn't understand the problem after reading the description.

Can I describe it as: A value is used as two different types in two basic blocks (or more). Although the bitcast of %11 (I guess optimizeBitCastFromPhi starts from here) can be removed with new set of PHIs, the old PHIs can't be removed due to its usage of type B in bb6. So we need to keep two set of values(registers) in many places even after DeSSA.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D57053





More information about the llvm-commits mailing list