bruno added inline comments.
================
Comment at: lib/Driver/ToolChains.cpp:3903
+ if (!Data.split('.').first.getAsInteger(10, MajorVersion)) {
+ switch (MajorVersion) {
+ case 5:
----------------
clang-format the `switch`: `case`'s should be aligned with it.
================
Comment at: lib/Driver/ToolChains.cpp:3917
+ }
+ } else {
+ return llvm::StringSwitch<Distro>(Data.split("\n").first)
----------------
You can remove the `else` and just `return llvm::StringSwitch...`
https://reviews.llvm.org/D25696
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
