[PATCH] D138711: AMDGPU: Remove BufferPseudoSourceValue
Jay Foad via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Nov 29 03:01:19 PST 2022
foad added inline comments.
================
Comment at: llvm/include/llvm/CodeGen/TargetLowering.h:1052-1054
+ // Fallback address space for use if ptrVal is nullptr. None means unknown
+ // address space.
+ Optional<unsigned> fallbackAddressSpace;
----------------
nhaehnle wrote:
> foad wrote:
> > "None means unknown address space" is a noble idea but I don't think it's providing any benefit in practice, since you only use this to initialize MachinePointerInfo, which has no concept of "unknown address space" - it just defaults to address space 0.
> How strongly do you feel about that? :)
Not very, but if you want to keep it then I think we need a TODO comment where you map "unknown address space" (in IntrinsicInfo) to "address space 0" (in MachinePointerInfo).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D138711/new/
https://reviews.llvm.org/D138711
More information about the llvm-commits
mailing list