[PATCH] D91950: [clang-format] Add BreakBeforeInlineASMColon configuration

Björn Schäpers via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jul 29 02:53:56 PDT 2021


HazardyKnusperkeks added a comment.

And format what clang-format says. Then it looks good to me.



================
Comment at: clang/unittests/Format/FormatTest.cpp:6250
+  verifyFormat("asm volatile(\"loooooooooooooooooooong\",\n"
+               "             : : val);",
+               Style);
----------------
Could you add for Never and Always the code from Above?

```
      "asm(\"movq\\t%%rbx, %%rsi\\n\\t\"\n"
      "    \"cpuid\\n\\t\"\n"
      "    \"xchgq\\t%%rbx, %%rsi\\n\\t\"\n"
      "    : \"=a\"(*rEAX), \"=S\"(*rEBX), \"=c\"(*rECX), \"=d\"(*rEDX)\n"
      "    : \"a\"(value));"
```


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

https://reviews.llvm.org/D91950



More information about the cfe-commits mailing list