[PATCH] D84822: Add documentation for target ID and ClangOffloadBundlerFormat

Yaxun Liu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 13 15:19:51 PDT 2020


yaxunl marked an inline comment as done.
yaxunl added inline comments.


================
Comment at: llvm/docs/AMDGPUUsage.rst:387
+
+  *Default*
+    Specified by omitting the target feature from the target ID.
----------------
arsenm wrote:
> Default is an extremely confusing name. This should reflect that it's the universally compatible mode. AnyMode? Either? Universal?
Another word I can think of is "General" or "Generic".

@t-tye What do you think? Thanks.


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

https://reviews.llvm.org/D84822



More information about the llvm-commits mailing list