uabelho added inline comments.
================
Comment at: clang/lib/Lex/InitHeaderSearch.cpp:336
- case llvm::Triple::Minix:
- AddGnuCPlusPlusIncludePaths("/usr/gnu/include/c++/4.4.3",
- "", "", "", triple);
----------------
brad wrote:
> uabelho wrote:
> > @brad :
> > I think this was the last use of the AddGnuCPlusPlusIncludePaths method,
> > should we remove it now or is it likely it will be used again?
> >
> > @brad :
> > I think this was the last use of the AddGnuCPlusPlusIncludePaths method,
> > should we remove it now or is it likely it will be used again?
>
> I noticed a function of the same name with the Darwin Driver.
Yeah Darwin defines and uses its own function with that name, but the one in
InitHeaderSearch is unused since this patch.
I don't know anything about this, just noticed that gcc warns about it being
unused.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D158715/new/
https://reviews.llvm.org/D158715
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
