[PATCH] D99233: [HIP] Add option --gpu-inline-threshold
Artem Belevich via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Apr 21 11:04:16 PDT 2021
tra added a comment.
> The planned new option for offloading will be a more generic solution, however, I expect it will take time to develop and be adopted.
Agreed. OK, let's use a hidden option until we have a better way of dealing with this.
================
Comment at: clang/include/clang/Driver/Options.td:954
+def gpu_inline_threshold_EQ : Joined<["--"], "gpu-inline-threshold=">,
+ Flags<[CC1Option]>,
+ HelpText<"Inline threshold for device compilation for HIP">;
----------------
This option is only handled at the top level, it does not need `CC1Option`. It does need `HelpHidden`.
Also, the option should probably be `-fgpu-inline-threshold=...` as it's a parameter tweak, and not something more serious, like `--offload-arch`.
Naming is hard. :-)
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D99233/new/
https://reviews.llvm.org/D99233
More information about the cfe-commits
mailing list