[PATCH] D130131: [HLSL] CodeGen hlsl cbuffer/tbuffer.

Chris Bieneman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Aug 12 17:30:07 PDT 2022


beanz added a comment.

Now that I'm seeing the code in D131370 <https://reviews.llvm.org/D131370>, I don't know that this is the right way to do things.

I think using address spaces like this is odd. Address spaces aren't really intended for use differentiating high level access types, but rather memory regions and properties of that memory. I feel like the use of address spaces in this change and in D131370 <https://reviews.llvm.org/D131370> just makes it more complicated and gets in the way.

In DXIL, since we don't really have raw memory load and stores, none of these address spaces mean anything. It is probably just cleaner to not use address spaces in the CodeGen here either.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130131



More information about the cfe-commits mailing list