[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