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

Pushpinder Singh via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Feb 16 08:28:53 PST 2021


pdhaliwal added a comment.

It is because of how addClangTargetOptions is invoked. In case of save-temps, it is being invoked for all the actions resulting in target cc1 call. That's why all these invocations have -emit-llvm-bc. I guess we need Action as an argument to addClangTargetOptions.

Also, it does not make sense for having assemble and backend action for amdgcn as linker is dependent directly on llvm IR. They will also come up redundantly in the -ccc-print-phases.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96769



More information about the cfe-commits mailing list