nemanjai accepted this revision. nemanjai added a comment. This revision is now accepted and ready to land.
LGTM. Thanks. ================ Comment at: clang/test/CodeGen/PowerPC/builtins-ppc-xlcompat.c:8 // RUN: -D__XL_COMPAT_ALTIVEC__ -target-cpu pwr8 | FileCheck %s -// RUN: %clang_cc1 -no-opaque-pointers -target-feature +altivec -target-feature +vsx \ +// RUN: %clang_cc1 -flax-vector-conversions=none -no-opaque-pointers -target-feature +altivec -target-feature +vsx \ // RUN: -triple powerpc64le-unknown-linux-gnu -emit-llvm %s -o - \ ---------------- lei wrote: > nemanjai wrote: > > I don't know why only one of the functions below has checks for this run > > line, but that needs to be fixed. Please add the `NOCOMPAT` checks on the > > other functions. > Can we address this on a followup patch? Since it's not related to the work > here. Absolutely. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D124093/new/ https://reviews.llvm.org/D124093 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits