daltenty added inline comments.
================
Comment at: clang/lib/Driver/ToolChains/AIX.h:26
+
+ bool hasIntegratedCPP() const override { return false; }
+
----------------
stevewan wrote:
> Xiangling_L wrote:
> > I saw a lot of other target also set `hasIntegratedCPP()` as false, but I
> > didn't find any explanation in documentation, so I am curious that what
> > this is about?
> I also failed to find useful resources that explains the purpose of this
> function. The main rationales of adding it were essentially that,
> 1. It's a pure virtual function in the base `Tool` class,
> 2. Most if not all of other targets had overridden this function such that it
> returns false.
>
> I'll leave this comment open, and see if someone could enlighten us.
Only "Compiler" tools set hasIntegratedCPP() to true. Looking into it, it seems
combineWithPreprocessor() uses this to decide whether the tool supports
preprocessor actions so it may fold them in to one step. I think we are safe
leaving this as is.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D69620/new/
https://reviews.llvm.org/D69620
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
