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

jansvoboda11 added a comment. In D103461#2862067 , @jansvoboda11 wrote: > In D103461#2841918 , @bmahjour > wrote: > >> @jansvoboda11 This change is causing the following LIT tests to fail on AIX: >> >> Clang :

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

jansvoboda11 added a comment. In D103461#2841918 , @bmahjour wrote: > @jansvoboda11 This change is causing the following LIT tests to fail on AIX: > > Clang :: ClangScanDeps/headerwithdirname.cpp > Clang :: ClangScanDeps/headerwithdirnamefollowedbyinc

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

dexonsmith added a comment. In D103461#2841918 , @bmahjour wrote: > @jansvoboda11 This change is causing the following LIT tests to fail on AIX: I think @jansvoboda11 is out this week (IIRC, back next week). A possible interim workaround would be to add

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

bmahjour added a comment. @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 def

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

jansvoboda11 added a comment. In D103461#2815913 , @dexonsmith wrote: > Okay, LGTM. (Sorry for the delay, I've been out.) > > In D103461#2793254 , @jansvoboda11 > wrote: > >> My reason for the FIXME is that we c

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

This revision was automatically updated to reflect the committed changes. Closed by commit rG669771cfe75b: [clang][deps] NFC: Preserve the original frontend action (authored by jansvoboda11). Changed prior to commit: https://reviews.llvm.org/D103461?vs=349229&id=351795#toc Repository: rG LLV

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

dexonsmith accepted this revision. dexonsmith added a comment. This revision is now accepted and ready to land. Okay, LGTM. (Sorry for the delay, I've been out.) In D103461#2793254 , @jansvoboda11 wrote: > My reason for the FIXME is that we could get ri