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

[Yaxun Liu via Phabricator via llvm-commits]

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.
----------------
t-tye wrote:
> yaxunl wrote:
> > 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.
> What about "any". It is short like "on" and "off" and captures what Matt suggests (a truncation of "anymode" which would imply "onmode" and "offmode" which seem a mouthful and no more obvious).
> 
> Thoughts?
"Any" sounds perfect. I will update the patch. Thanks.


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

https://reviews.llvm.org/D84822