shchenz added inline comments.
================ Comment at: clang/lib/Driver/ToolChains/AIX.cpp:232 + path::remove_filename(P); + addSystemInclude(DriverArgs, CC1Args, P); } ---------------- Can we use `path::parent_path(P)` directly in `addSystemInclude()`? `remove_filename()` sounds like `ppc_wrappers` is a file. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D129461/new/ https://reviews.llvm.org/D129461 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits