rampitec added a comment. In D112041#3073637 <https://reviews.llvm.org/D112041#3073637>, @hliao wrote:
> In D112041#3073560 <https://reviews.llvm.org/D112041#3073560>, @rampitec > wrote: > >> Is there anything to remove assume() call after address space is inferred? >> We do not need it anymore. > > along with a few other intrinsics, assume intrinsic is discarded in SDAG and > GISel. We may want to discard these earlier for the sake of Value::hasOneUse(). These are really not needed after casts are inserted. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D112041/new/ https://reviews.llvm.org/D112041 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
