[PATCH] D91428: Add support for multiple program address spaces

Alexander Richardson via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 23 12:12:45 PST 2020


arichardson added a comment.

The change to rename getProgramAddressSpace getDefaultProgramAddressSpace seems fine to me since it matches the GlobalsAddressSpace.

By the way, your test already seems to work (if you add an explicit `call addrspace(1) void %ref()` for llc: https://godbolt.org/z/bqYMdE


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D91428



More information about the cfe-commits mailing list