[PATCH] D53417: [Clang][PowerPC] Choose a better candidate in overload function call if there is a compatible vector conversion instead of ambiguous call error

Zixuan Wu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Oct 19 19:06:13 PDT 2018


wuzish added inline comments.


================
Comment at: clang/test/Sema/altivec-generic-overload.c:3
+
+typedef signed char __v4sc __attribute__((__vector_size__(16)));
+typedef unsigned char __v4uc __attribute__((__vector_size__(16)));
----------------
hubert.reinterpretcast wrote:
> `__v4sc` is suspicious. The most consistent naming I have seen is `v16i8`, `v16u8`, `v8i16`, ...
> Do we need the double underscores?
> 
Because it's generic gcc vector type instead of altivec type so I choose the naming style.


Repository:
  rC Clang

https://reviews.llvm.org/D53417





More information about the cfe-commits mailing list