[PATCH] D76795: [HIP] Change default --gpu-max-threads-per-block value to 1024

[Yaxun Liu via Phabricator via cfe-commits]

yaxunl marked 2 inline comments as done.
yaxunl added inline comments.


================
Comment at: clang/lib/CodeGen/TargetInfo.cpp:8123
+    // --gpu-max-threads-per-block=n or its default value for HIP.
+    const unsigned OpenCLMaxWorkGroupSize = 256;
+    const unsigned MaxWorkGroupSize =
----------------
b-sumner wrote:
> I'd like to see the word default, e.g. OpenCLDefaultMaxWorkGroupSize, used more since that is what this is about.  Ideally the option would have been named gpu-default-max-threads-per-block, but I suppose I can see why it was shortened.
changed variable names.


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

https://reviews.llvm.org/D76795