[PATCH] D80144: [clang-format] @lefticus just taught the world how to use [[unlikely]] but we forgot to teach clang-format

Jake Merdich via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon May 18 17:22:20 PDT 2020


JakeMerdichAMD added a comment.

This is a great improvement in readability. I think this will get in here before it does for clang proper :D

Likely/unlikely also seem to be supported on while, do-while, and for loops (case statements too, but that's not relevant). Should we also apply identical logic there? Far less useful/common than plain if's so it's not a blocker, but it's best to be consistent and I can imagine some decent use cases.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D80144/new/

https://reviews.llvm.org/D80144





More information about the cfe-commits mailing list