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

[Eric Christopher via llvm-commits]

Agreed.

On Fri, Jul 12, 2019, 9:54 AM Nikita Popov via Phabricator <
reviews at reviews.llvm.org> wrote:

> 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
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190712/72947c0c/attachment-0001.html>