[PATCH] D112041: [InferAddressSpaces] Support assumed addrspaces from addrspace predicates.

[Stanislav Mekhanoshin via Phabricator via llvm-commits]

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