[PATCH] D45212: [HIP] Let CUDA toolchain support HIP language mode and amdgpu

John McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Apr 6 20:23:31 PDT 2018


rjmccall added a comment.

This looks okay to me, but I think it would better if someone with more expertise in the design of the driver and frontend code could review this.



================
Comment at: lib/Driver/Driver.cpp:2267
+        if ((IA->getType() != types::TY_CUDA) &&
+            IA->getType() != types::TY_HIP) {
           // The builder will ignore this input.
----------------
The extra parens are weird here.


https://reviews.llvm.org/D45212





More information about the cfe-commits mailing list