[PATCH] D76693: [Sema][SVE] Allow ?: to select between SVE types in C

2020-03-25 Thread Richard Sandiford via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG578fb2501a66: [Sema][SVE] Allow ?: to select between SVE types in C (authored by rsandifo-arm). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D76693/new/

[PATCH] D76693: [Sema][SVE] Allow ?: to select between SVE types in C

2020-03-24 Thread Eli Friedman via Phabricator via cfe-commits
efriedma accepted this revision. efriedma added a comment. This revision is now accepted and ready to land. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D76693/new/ https://reviews.llvm.org/D76693

[PATCH] D76693: [Sema][SVE] Allow ?: to select between SVE types in C

2020-03-24 Thread Richard Sandiford via Phabricator via cfe-commits
rsandifo-arm added a comment. In D76693#1939866 , @efriedma wrote: > Do we want to allow stuff like `x ? (svint8_t)0 : (signed char)0` `(svint8_t)0` is invalid as thing stand. The intention is for the types to be opaque and only used via intrinsics,

[PATCH] D76693: [Sema][SVE] Allow ?: to select between SVE types in C

2020-03-24 Thread Eli Friedman via Phabricator via cfe-commits
efriedma added a comment. Do we want to allow stuff like `x ? (svint8_t)0 : (signed char)0` Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D76693/new/ https://reviews.llvm.org/D76693 ___ cfe-commits

[PATCH] D76693: [Sema][SVE] Allow ?: to select between SVE types in C

2020-03-24 Thread Richard Sandiford via Phabricator via cfe-commits
rsandifo-arm created this revision. rsandifo-arm added reviewers: sdesmalen, efriedma, rovka, rjmccall. Herald added subscribers: cfe-commits, psnobl, rkruppe, tschuett. Herald added a reviewer: rengolin. Herald added a project: clang. When compiling C, a ?: between two values of the same SVE