Skip to content

Commit

Permalink
Enable the ARM Neon intrinsics test by default.
Browse files Browse the repository at this point in the history
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.)

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@304302 91177308-0d34-0410-b5e6-96231b3b80d8
  • Loading branch information
alexfh committed May 31, 2017
1 parent 1cb6614 commit 2bb177b
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions test/CodeGen/arm_neon_intrinsics.c
Original file line number Diff line number Diff line change
Expand Up @@ -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(
Expand Down

0 comments on commit 2bb177b

Please sign in to comment.