[PATCH] D103461: [clang][deps] NFC: Preserve the original frontend action

Wed Jul 7 08:54:23 PDT 2021

jansvoboda11 added a comment.

In D103461#2841918 <https://reviews.llvm.org/D103461#2841918>, @bmahjour wrote:

> @jansvoboda11 This change is causing the following LIT tests to fail on AIX:
>
>   Clang :: ClangScanDeps/headerwithdirname.cpp
>   Clang :: ClangScanDeps/headerwithdirnamefollowedbyinclude.cpp
>
> The reason seems to be related to the fact that `-fno-integrated-as` is on by default on that platform. I get the same failure on Linux if I change the "compilation database" file to add `-fno-integrated-as` to the "command" line:
>
>   > /build_llvm/bin/clang-scan-deps -compilation-database ./headerwithdirname.cpp.tmp.cdb -j 1
>   > Error while scanning dependencies for /build_llvm/tools/clang/test/ClangScanDeps/Output/headerwithdirname.cpp.tmp.dir/headerwithdirname_input.cpp:
>   error: unable to handle compilation, expected exactly one compiler job in ' "clang" "-cc1" "-triple" "powerpc64le-unknown-linux-gnu" "-S" ...;  "/usr/bin/as" "-a64" "-mppc64" "-mlittle-endian" "-mpower8" "-I" "/build_llvm/tools/clang/test/ClangScanDeps/Output/headerwithdirname.cpp.tmp.dir" "-I" "/build_llvm/tools/clang/test/ClangScanDeps/Output/headerwithdirname.cpp.tmp.dir/foodir" "-I" "Inputs" "-o" "headerwithdirname_input.o" "/tmp/headerwithdirname_input-2e0050.s"; '

Thanks for the report and the reproducer. I'll try to get a fix ready tomorrow.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103461