[PATCH] D46471: [HIP] Add hip offload kind

John McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri May 4 20:53:56 PDT 2018


rjmccall added a comment.

Otherwise LGTM.



================
Comment at: lib/Driver/Compilation.cpp:201
+  // not compiled again if there are already failures. It is OK to abort the
+  // CUDA pipeline on errors.
+  if (A->isOffloading(Action::OFK_Cuda) || A->isOffloading(Action::OFK_HIP))
----------------
Mentioning only CUDA in the second clause makes me wonder whether it's *only* okay to abort a CUDA pipeline, not a HIP one.  That is presumably not your intent.  You could just drop "CUDA" there.


https://reviews.llvm.org/D46471





More information about the cfe-commits mailing list