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

[Jan Svoboda via Phabricator via cfe-commits]

jansvoboda11 added a comment.

In D103461#2862067 <https://reviews.llvm.org/D103461#2862067>, @jansvoboda11 wrote:

> 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.

Fixed here: https://reviews.llvm.org/D105695


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103461