beanz added inline comments.

================
Comment at: clang/lib/CodeGen/CGHLSLRuntime.h:73
   CodeGenModule &CGM;
   uint32_t ResourceCounters[static_cast<uint32_t>(
       hlsl::ResourceClass::NumClasses)] = {0};
----------------
The `ResourceCounters` here was a stand-in for allocating resource indices.

If you have a different path for allocating these, we shouldn't need the 
counter anymore. I'm a little concerned that it looks like you're not 
allocating these in code generation. Can you explain how you intend to allocate 
indices? Is there a reason not to do this in code gen?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130951

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to