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

Tue Feb 16 08:08:48 PST 2021

jdoerfert added a comment.

In D96769#2565751 <https://reviews.llvm.org/D96769#2565751>, @pdhaliwal wrote:

> emit-llvm-bc does not correctly solve the problem. It works because [input, compile, assemble, backend] actions collapse to a single action by driver. This single command handles emit-llvm-bc properly. But when save-temps is specified, this collapsing does not happen which messes up command line flags of the jobs and hence the output, for e.g., preprocessor command also has -emit-llvm-bc.

Is this the intended behavior of -save-temps + -emit-llvm-bc or an accident? What exactly happens when both are specified and what is the problem. Maybe we should fix that instead of working around it?

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96769