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