[PATCH] D61717: Fix arm_neon.h to be clean under -fno-lax-vector-conversions.

Richard Smith - zygoloid via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Sep 17 13:34:28 PDT 2019


rsmith added a comment.

In D61717#1668980 <https://reviews.llvm.org/D61717#1668980>, @SjoerdMeijer wrote:

> Do you perhaps have a test case or error that I can look at? Perhaps I or someone else here can help out a bit here.


You can reproduce the problem with, for example:

  echo '#include <arm_neon.h>' | clang -target arm64-linux-gnu -arch +neon -fsyntax-only -x c - -fno-lax-vector-conversions

I've cleaned up all our other intrinsics headers to be clean under at least `-flax-vector-conversions=integer`. I'll be switching the default for all targets other than ARM NEON to that imminently. (It'll take a bit more work to be able to switch to `-fno-lax-vector-conversions` by default, but getting there should be our goal.)

I've filed PR43341 for this issue.


Repository:
  rC Clang

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

https://reviews.llvm.org/D61717





More information about the cfe-commits mailing list