MaskRay added a comment. Thanks for doing this!
================ Comment at: clang/lib/Driver/ToolChain.cpp:553 + // armeb (big endian) is left out here on purpose so that we don't try to use + // little endian libraries in that case. ---------------- The comments can be combined. First state what this block does, then state that armeb is left out for a reason. An example may be more descriptive than some words. Give an example of `getTriple()` and Arch values? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D140011/new/ https://reviews.llvm.org/D140011 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
