[PATCH] D60351: [builtins] Reformat builtins with clang-format

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 12 09:54:55 PDT 2019


nikic added inline comments.
Herald added a subscriber: wuzish.


================
Comment at: compiler-rt/trunk/lib/builtins/arm/sync-ops.h:37
+  DEFINE_COMPILERRT_THUMB_FUNCTION(__sync_fetch_and_##op)                      \
+  push{r4, r5, r6, lr};                                                        \
+  dmb;                                                                         \
----------------
It looks like this reformatted to illegal assembly:

```
compiler-rt/lib/builtins/arm/sync_fetch_and_add_8.S:21: Error: bad instruction `push{r4, r5,r6,lr}'
```

The whitespace between `push` and `{` likely needs to be preserved.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D60351





More information about the llvm-commits mailing list