[PATCH] D52093: [ToolChains] Linux.cpp: Use the same style across all all multi-arch includes. NFC.
Kristina Brooks via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Sat Sep 15 11:38:47 PDT 2018
kristina added a comment.
Yeah but what about the rest of them, most are completely on their own line, some no newlines, mipsel64 was split across two lines while others weren't, and switch is meant to have default as the topmost case IIRC. I mean I'm if you don't think it's more readable that way I'll abandon it. Though IMHO reading wise it's much easier to read as a list down with separators than as a mix of various newline/no-newline styles.
Repository:
rC Clang
https://reviews.llvm.org/D52093
More information about the cfe-commits
mailing list