jasonliu added inline comments.
================ Comment at: clang/lib/Driver/ToolChains/AIX.h:19 + +/// aix -- Directly call system default and linker. +// TODO: Enable direct call to system default assembler. ---------------- remove "and". ================ Comment at: clang/lib/Driver/ToolChains/AIX.h:36 + +} // end namespace aix. + ---------------- Sorry, I don't think we need '.' for any of the "end namespace ..." comment. ================ Comment at: clang/lib/Driver/ToolChains/AIX.h:63 + +#endif // LLVM_CLANG_LIB_DRIVER_TOOLCHAINS_AIX_H. ---------------- We don't need '.' here as well. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68340/new/ https://reviews.llvm.org/D68340 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
