[Openmp-commits] [PATCH] D156387: [OpenMP][Docs] Added offloading command line reference to OpenMP FAQ

[Jan-Patrick Lehr via Phabricator via Openmp-commits]

jplehr added a subscriber: jhuber6.
jplehr added inline comments.


================
Comment at: openmp/docs/SupportAndFAQ.rst:477-478
+""""""""""""""""""""""""""""""""""""""""""""""""""""
+Compile the target regions for the host only. All target regions will be 
+compiled for both host and device if not specified.
+
----------------
AntonRydahl wrote:
> jplehr wrote:
> > Flip the flags or the sentences, so that the flags and their meaning are in the same order.
> Thank you for pointing this out! I thought the two flags had the same meaning as they had the same description in the Clang command line reference. 
> 
> ```
> clang --help | grep offload-host-device
>   --offload-host-device   Only compile for the offloading host.
> clang --help | grep offload-host-only
>   --offload-host-only     Only compile for the offloading host.
> ```
> 
> What exactly is the difference between the two flags?
To be honest, I only guessed from their name, hopefully someone else knows. My interpretation:
``--offload-host-device`` -> Compile for host and device
``--offload-host-only`` -> Compile for host only

Let's try summoning @jdoerfert or @jhuber6 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D156387