[PATCH] D19369: VirtRegMap.cpp should not just remove an identity copy which also has a impl-def of a super-reg

Quentin Colombet via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 21 10:54:50 PDT 2016


Also, side question, how do we end up with this representation?
Do we use an undef lane or something, that’s why the verifier complains?

> On Apr 21, 2016, at 9:47 AM, Quentin Colombet <qcolombet at apple.com> wrote:
> 
> qcolombet added a subscriber: qcolombet.
> qcolombet added a comment.
> 
> We could fix MachineInstr:: isIdentityCopy instead. Indeed, the next optimization that removes them will hit the same issue. Moreover, I do not think we are supposed to have implicit_def around after TwoAddrPass (not entirely sure about that though).
> 
> Q.
> 
> 
> http://reviews.llvm.org/D19369
> 
> 
> 



More information about the llvm-commits mailing list