r304302 - Enable the ARM Neon intrinsics test by default.

Alexander Kornienko via cfe-commits cfe-commits at lists.llvm.org
Wed May 31 07:35:50 PDT 2017


Author: alexfh
Date: Wed May 31 09:35:50 2017
New Revision: 304302

URL: http://llvm.org/viewvc/llvm-project?rev=304302&view=rev
Log:
Enable the ARM Neon intrinsics test by default.

The test being marked 'REQUIRES: long-tests' doesn't make sense. It's not the
first time the test is broken without being noticed by the committer. If the
test is too long, it should be shortened, split in multiple ones or removed
altogether. Keeping it as is is actively harmful.
(BTW, on my machine `ninja check-clang` takes 90-92 seconds with and without
this test. The difference in times is below the spread caused by random
factors.)

Modified:
    cfe/trunk/test/CodeGen/arm_neon_intrinsics.c

Modified: cfe/trunk/test/CodeGen/arm_neon_intrinsics.c
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/CodeGen/arm_neon_intrinsics.c?rev=304302&r1=304301&r2=304302&view=diff
==============================================================================
--- cfe/trunk/test/CodeGen/arm_neon_intrinsics.c (original)
+++ cfe/trunk/test/CodeGen/arm_neon_intrinsics.c Wed May 31 09:35:50 2017
@@ -3,8 +3,6 @@
 // RUN:  -disable-O0-optnone -emit-llvm -o - %s \
 // RUN:  | opt -S -mem2reg | FileCheck %s
 
-// REQUIRES: long-tests
-
 #include <arm_neon.h>
 
 // CHECK-LABEL: @test_vaba_s8(




More information about the cfe-commits mailing list