[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