myhsu added inline comments.
================ Comment at: clang/lib/Driver/ToolChains/Gnu.cpp:2101 + static const char *const M68kLibDirs[] = {"/lib"}; + static const char *const M68kTriples[] = { ---------------- MaskRay wrote: > The second `const ` is redundant I don't think so...first, `const char *const` means "a pointer points to const char, and you can't modify this pointer", which make sense in this context. Second, all the surrounding code here are using `const char *const`. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88394/new/ https://reviews.llvm.org/D88394 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits