[PATCH] D103348: [AMDGPU] Add maximum NSA size limit ISA feature

Carl Ritson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 4 18:43:51 PDT 2021


critson marked 3 inline comments as done.
critson added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/GCNSubtarget.h:139
   bool HasNSAEncoding;
+  int NSAMaxSize;
   bool GFX10_BEncoding;
----------------
foad wrote:
> Is there a reason this can't be unsigned?
The tablegen part needs to be int, but this can be unsigned.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103348



More information about the llvm-commits mailing list