[PATCH] D75675: [clang-format] do not insert spaces around inline asm symbolic names

Hans Wennborg via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Mar 5 04:56:45 PST 2020


hans added a comment.

Very nice, thanks!



================
Comment at: clang/lib/Format/TokenAnnotator.cpp:502
         Left->Type = TT_ObjCMethodExpr;
+      } else if (Style.isCpp() && InsideInlineASM) {
+        Left->Type = TT_InlineASMSymbolicNameLSquare;
----------------
Is the Style.isCpp() check necessary here?

If it's necessary, maybe that's because the setting of InsideInlineASM should be guarded by Style.isCpp() instead?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75675





More information about the cfe-commits mailing list