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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits