peterwaller-arm accepted this revision. peterwaller-arm added a comment. This revision is now accepted and ready to land.
LGTM, modulo that the require lines look like they need fixing. ================ Comment at: clang/test/CodeGen/aarch64-sve-vector-bitwise-ops.c:7 + +// AND + ---------------- Needs a requires line? ================ Comment at: clang/test/Sema/aarch64-sve-vector-bitwise-ops.c:2-3 +// RUN: %clang_cc1 -verify -triple aarch64-none-linux-gnu -target-feature +sve -fallow-half-arguments-and-returns -fsyntax-only %s + +// REQUIRES: aarch64-registered-target || arm-registered-target + ---------------- Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D121119/new/ https://reviews.llvm.org/D121119 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
