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

[Alexander Richardson via Phabricator via llvm-commits]

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