[PATCH] D23896: [InstCombine] Try to resubmit the combine of A->B->A BitCast and fix for pr27996

Guozhi Wei via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 9 13:44:03 PDT 2016


Carrot added a comment.

In https://reviews.llvm.org/D23896#537933, @evstupac wrote:

> Hi,
>
> > InstCombine is supposed to run to fix-point
>
>
> Have you addressed David's comment?
>
> Am I right that current fix-point is when all CI users are memory instructions?


Yes, optimizeBitCastFromPhi only generates new BitCast for load/store instructions, and now it will not work on BitCast used by MemOp only, so it will never work on the BitCast generated by itself.


https://reviews.llvm.org/D23896





More information about the llvm-commits mailing list