[PATCH] D23172: IfConversion: Add implicit uses for live subregisters

[Quentin Colombet via llvm-commits]

qcolombet added a comment.

Hi Krzysztof,

We were discussing similar liveness issues in https://reviews.llvm.org/D17257 with Jonas and I believe the right way to fix that would be to switch the live-in sets to RegUnit. The implicit operand would work but they look like a workaround to the actual problem to me.
Adding the RegUnit instead of the PhysReg is easy and could be hidden in the addLiveIn method, removing them is a different story.

Cheers,
-Quentin


Repository:
  rL LLVM

https://reviews.llvm.org/D23172