[PATCH] D138560: [lld][LTO] Add assembly output to LTO save-temps

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Dec 17 12:31:18 PST 2022


MaskRay added a comment.

(I was out of town for ~2 weeks.)

Thanks for abandoning this. Running passes twice in LLVMLTO is definitely weird, even if it appears to be the simplest approach satisfying your goal.
If you want both `.s` and `.o`, I wish that you think about doing this as whatever tool which wraps clang driver instead of doing it in the clang driver.
Regular non-GPU clang driver code doesn't allow emit both `.s` and `.o` in one run, either.
I don't track GPU changes that closely, so if you end up doing that, I might not see or object in the end.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D138560



More information about the llvm-commits mailing list