[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