[PATCH] D121294: [AArch64][SVE][Clang] Fix crash for incorrect svptrue and svcnt parameters

Matt Devereau via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Mar 9 08:22:10 PST 2022


MattDevereau added a comment.

In D121294#3370085 <https://reviews.llvm.org/D121294#3370085>, @sdesmalen wrote:

> This is missing tests for svundef, svrdffr, svsetffr and svpfalse?

@sdesmalen Only svcnt and svptrue cause the crash. This might be because of extra values added by the `[IsAppendSVALL]` TypeFlag in `arm_sve.td`



================
Comment at: clang/test/CodeGen/aarch64-sve-intrinsics/negative/acle_sve_svptrue.c:2
+// REQUIRES: aarch64-registered-target
+
+// RUN: %clang_cc1 -triple aarch64-none-linux-gnu -target-feature +sve -fallow-half-arguments-and-returns -fsyntax-only -verify %s
----------------
sdesmalen wrote:
> Is it possible to use the `update_cc_test_checks.py` script for these tests?
Deleting `// expected-error-re at +1...` and running `update_cc_test_checks.py` did not generate anything in its place in either of the tests


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D121294



More information about the cfe-commits mailing list