[PATCH] D142222: [PowerPC] Remove the lax warning for explicit casts

Nemanja Ivanovic via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jan 20 08:38:02 PST 2023


nemanjai accepted this revision.
nemanjai added a comment.
This revision is now accepted and ready to land.

LGTM other than the missing template test.



================
Comment at: clang/test/Parser/lax-conv.cpp:4
+// RUN: %clang_cc1 -triple=powerpc64-ibm-aix -target-feature +altivec -target-feature -vsx -fsyntax-only -verify=expected,aix %s
+
+void dummy(vector unsigned int a);
----------------
Can you also add one template test?
```
template <typename VEC> VEC __attribute__((noinline)) test(vector unsigned char a, vector unsigned char b) {
  return (VEC)(a * b);
}

vector unsigned char test1(vector unsigned char a, vector unsigned char b) {
  return test<vector unsigned char>(a, b);
}

vector unsigned long long test2(vector unsigned char a, vector unsigned char b) {
  return test<vector unsigned long long>(a, b);
}
```


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142222



More information about the cfe-commits mailing list