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

Michael Liao via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 8 12:31:32 PDT 2019


hliao added a comment.

In D68587#1700178 <https://reviews.llvm.org/D68587#1700178>, @tra wrote:

> TL; DR; 
>  +1 to formalizing how we want -M*/-E/-S/-emit-llvm/-fsyntax-only to behave. 
>  OK with -M/-E/-S defaulting to host, and erroring out if applied to multiple sub-compilations.
>  I'm still convinced that the tooling issue with multiple subcompilations is orthogonal to this change and should be handled in libclang and that -fsyntax-only should not default to one sub-compilation.


I am starting to fix clang tooling issues found in both driver (https://reviews.llvm.org/D68652) and tooling parts. Will submit tooling part for review soon. After that, we may nail down the details on expected output from -M*/-E and etc.


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