steakhal added inline comments.

================
Comment at: clang/lib/StaticAnalyzer/Core/ExprEngineC.cpp:523
       }
+        LLVM_FALLTHROUGH;
       // Various C++ casts that are not handled yet.
----------------
steakhal wrote:
> NoQ wrote:
> > Dunno, is this the intended formatting for fallthrough? Seems kinda off 
> > near those braces.
> I agree.
> I'm using clang-format-10 at the moment.
> Formats the diff automatically on save, so I assume it's the 'recommended' 
> place.
It seems like the most recent clang-format behaves the same.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105125

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D105125: [analyzer]... Balázs Benics via Phabricator via cfe-commits
    • [PATCH] D105125: [anal... Balázs Benics via Phabricator via cfe-commits

Reply via email to