[PATCH] D138563: [IntrisicEmitter] Allow intrinsic types to be dependent on the data layout

Alexander Richardson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 7 07:14:53 PST 2022


arichardson abandoned this revision.
arichardson added a comment.

In D138563#3973909 <https://reviews.llvm.org/D138563#3973909>, @nikic wrote:

> FWIW, we already allow allocas to have an address space that differs from the default alloca address space. IIRC it was added for some wasm use case.

I did not realize that allocas were allowed in multiple address spaces. In that case the only intrinsics that benefit from this are the `returnaddress` ones and those could conceivable also be wanted with multiple address spaces (if there are functions in different address spaces).
Will upload reviews to overload the intrinisics instead.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D138563



More information about the llvm-commits mailing list