[PATCH] D154123: [HIP] Start document HIP support by clang

[Siu Chi Chan via Phabricator via cfe-commits]

scchan added inline comments.


================
Comment at: clang/docs/HIPSupport.rst:33
+
+   clang -c --offload-arch=gfx906 test.hip -o test.o
+
----------------
can we add -xhip to enable HIP for files without the .hip file ext?


================
Comment at: clang/docs/HIPSupport.rst:56
+Environment Variables
+=====================
+
----------------
I'd suggest adding a section to cover compiler options to pass various hip/rocm related paths and that using the compiler options would be a much more preferable alternative.


================
Comment at: clang/docs/HIPSupport.rst:87
+   * - ``__HIPCC__``
+     - This macro indicates that the code is being compiled using the HIP compiler.
+   * - ``__HIP_MEMORY_SCOPE_SINGLETHREAD``
----------------
I think this macro is essentially an alias to __HIP__?


================
Comment at: clang/docs/HIPSupport.rst:99
+   * - ``__HIP_DEVICE_COMPILE__``
+     - This macro is defined when the code is being compiled for a HIP device.
+   * - ``__HIP_NO_IMAGE_SUPPORT``
----------------
It might be helpful to provide more context here as to how a HIP source file gets compiled (once for the host and then once for each device arch).  Also I'd suggest moving this up closer to __HIP__ given they are related.


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

https://reviews.llvm.org/D154123