hubert.reinterpretcast requested changes to this revision. hubert.reinterpretcast added inline comments. This revision now requires changes to proceed.
================ Comment at: clang/lib/Sema/SemaOverload.cpp:3920 + + // Prefer a compatible vector conversion to lax vector conversion + // For example: ---------------- s/to/over a/; ================ Comment at: clang/lib/Sema/SemaOverload.cpp:3941 + return ImplicitConversionSequence::Better; + } + ---------------- This seems to duplicate the bug described here in https://bugs.llvm.org/show_bug.cgi?id=39361. ``` typedef unsigned int GccType __attribute__((__vector_size__(16))); typedef __vector unsigned int AltiVecType; typedef float GccOtherType __attribute__((__vector_size__(16))); char *f(GccOtherType, int); template <typename T> int f(AltiVecType, T); template <typename T> int g(AltiVecType, T); char *g(GccOtherType, int); bool zip(GccType v) { return f(v, 0) == g(v, 0); } ``` Repository: rC Clang https://reviews.llvm.org/D53417 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
