[PATCH] D155213: [HIP] Add `-fno-hip-uniform-block`
Matt Arsenault via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Jul 14 10:42:55 PDT 2023
arsenm added inline comments.
================
Comment at: clang/include/clang/Driver/Options.td:1092
ShouldParseIf<hip.KeyPath>;
+defm hip_uniform_block : BoolFOption<"hip-uniform-block",
+ LangOpts<"HIPUniformBlock">, DefaultTrue,
----------------
scchan wrote:
> arsenm wrote:
> > Can we avoid adding yet another language flag for something that's reusable for everything? Is there an --offload-<something> ?
> Don't we need a different default value for some languages like OpenCL?
Yes, but opencl already has a spec'd flag for this. If we're making up a new one, it could be something generic that aliases the opencl one in that case. Plus the +/- value of a new flag should work (the CL one only goes in one direction)
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D155213/new/
https://reviews.llvm.org/D155213
More information about the cfe-commits
mailing list