[PATCH] D96769: [OpenMP][AMDGPU] Skip backend and assemble phases for amdgcn

[Jon Chesterfield via Phabricator via cfe-commits]

JonChesterfield added a comment.

Replacing `CC1Args.push_back("-emit-llvm-bc");` with `CC1Args.push_back("-emit-llvm-bc");` as suggested on the call does not work. This hook is downstream of the clang driver, so all it does under save temps is lead to `clang -E -emit-llvm`, which generated llvm as requested, that cannot be fed into `clang -x cpp-output`.

The handling of `clang -save-temps` that strips out `emit-llvm` from the preprocessor pass runs before this.

I do not want to change the semantics of emit-llvm, emit-llvm-bc, or save-temps to help out the amdgcn openmp target. I can see that breaking lots of other users of clang.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96769