[PATCH] D81670: [TTI] Expose isNoopAddrSpaceCast from TLI.[SROA] Teach SROA to recognize no-op addrspacecast.

Matt Arsenault via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jun 11 14:20:30 PDT 2020


arsenm added a comment.

In D81670#2088304 <https://reviews.llvm.org/D81670#2088304>, @hliao wrote:

> In D81670#2087974 <https://reviews.llvm.org/D81670#2087974>, @arsenm wrote:
>
> > We should instead allow bitcast to perform no-op addrspacecasts
>
>
> That may be a little bit challenging as so far no-op `addrspacecast` is target-specific. There may be no TTI available when `bitcast` is constructed.


Yes, that is the point. You don't need TTI to use it. Addrspace bitcast will be just a dumb bit reinterpret, not something that may change the value like addrspacecast


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D81670/new/

https://reviews.llvm.org/D81670





More information about the cfe-commits mailing list