[PATCH] D68587: [hip] Assume host-only compilation if the final phase is ahead of `backend`.

Artem Belevich via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Oct 7 12:52:15 PDT 2019


tra added a comment.

I'm fine with this for -E/-M,

I would argue that with `-fsyntax-only` we want to know whether our source code, which is common for all sub-compilations, has syntactic errors. 
The way we compile HIP & CUDA sources, some of the errors will only be reported on one side of the compilation. 
So, in order to make sure there are no syntax errors, we need to perform *all* sub-compilations with `-fsyntax-only`.

E.g. it would be rather surprising to see the compilation succeeding with `-fsyntax-only`, but then fail with a syntax error somewhere on device side during a real compilation.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D68587





More information about the cfe-commits mailing list