[Mesa-dev] [PATCH 1/3] vc4: Make Neon inline assembly clang compatible

Eric Anholt eric at anholt.net
Tue Jan 31 22:05:35 UTC 2017


Rob Herring <robh at kernel.org> writes:

> clang throws an error on "%r2" and similar. I couldn't find any
> documentation on what "%r?" is supposed to mean and I've never seen any
> use like that as far as I remember. The parameter is supposed to be
> cpu_stride and just %2/%3 should be sufficient.
>
> There's no need for trailing ";" either, so remove those, too.
>
> Signed-off-by: Rob Herring <robh at kernel.org>
> ---
> Eric,
>
> I *think* this is correct, but completely untested other than compiled 
> on Android.

Thanks!  I've done a build of the series for armv7 and confirmed no
change in the built output.

I should really go make a travis build for armv7 and arm64 to help
reduce trouble like this in the future.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 832 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20170131/e475fb8d/attachment.sig>


More information about the mesa-dev mailing list