[PATCH] D156886: [CUDA][HIP] Reorganize options for documentation
Yaxun Liu via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Aug 4 14:11:34 PDT 2023
yaxunl marked 2 inline comments as done.
yaxunl added inline comments.
================
Comment at: clang/include/clang/Driver/Options.td:155
+def offload_Group : OptionGroup<"<offload group>">, Group<f_Group>,
+ DocName<"Common Offloading flags">;
+
----------------
MaskRay wrote:
> The existing `flags` uses are misnomer (`Flag` does not accept values). Perhaps use `options` for new option groups.
will do
================
Comment at: clang/include/clang/Driver/Options.td:1021
+def offload_host_device : Flag<["--"], "offload-host-device">, Flags<[FlangOption]>,
+ HelpText<"Only compile for the offloading host.">;
+
----------------
jhuber6 wrote:
> Can you fix this copy paste error while you're here?
It has been fixed in the trunk.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D156886/new/
https://reviews.llvm.org/D156886
More information about the cfe-commits
mailing list