[PATCH] D67678: PR17164: Change clang's default behavior from -flax-vector-conversions=all to -flax-vector-conversions=integer.

Steven Wu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jan 22 11:37:40 PST 2020


steven_wu added a comment.

@rsmith This also breaks macOS SDK. Can we revert this as we discuss a less aggressive option?

Here is an example in macOS SDK (from /usr/include/simd/logic.h):

  static inline SIMD_CFUNC simd_bool simd_any(simd_int2 x) {
    return (_mm_movemask_ps(simd_make_int4_undef(x)) & 0x3);
  }

This change will cause problem for people using TOT clang with macOS SDK. Maybe we can start with warning or provide a toolchain overwrite for this?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D67678





More information about the cfe-commits mailing list