[PATCH] D105125: [analyzer][NFC] Inline ExprEngine::handleLVectorSplat()

[Balázs Benics via Phabricator via cfe-commits]

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