Skip to content

Commit

Permalink
Again test fixes.
Browse files Browse the repository at this point in the history
  • Loading branch information
bluescarni committed Nov 19, 2023
1 parent 8f20021 commit ef89618
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 15 deletions.
8 changes: 3 additions & 5 deletions test/acosh.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -311,7 +311,9 @@ TEST_CASE("vfabi float")
REQUIRE(outs[2] == approximately(std::acosh(1.3f)));
REQUIRE(outs[3] == approximately(std::acosh(1.4f)));

#if defined(HEYOKA_WITH_SLEEF)
// NOTE: autovec with external scalar functions seems to work
// only since LLVM 16.
#if defined(HEYOKA_WITH_SLEEF) && LLVM_VERSION_MAJOR >= 16

const auto &tf = detail::get_target_features();

Expand All @@ -334,15 +336,11 @@ TEST_CASE("vfabi float")
REQUIRE(count == 5u);
}

#if LLVM_VERSION_MAJOR >= 16

// NOTE: LLVM16 is currently the version tested in the CI on arm64.
if (tf.aarch64) {
REQUIRE(count == 5u);
}

#endif

// NOTE: currently no auto-vectorization happens on ppc64 due apparently
// to the way the target machine is being set up by orc/lljit (it works
// fine with the opt tool). When this is resolved, we can test ppc64 too.
Expand Down
8 changes: 3 additions & 5 deletions test/asinh.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -311,7 +311,9 @@ TEST_CASE("vfabi float")
REQUIRE(outs[2] == approximately(std::asinh(1.3f)));
REQUIRE(outs[3] == approximately(std::asinh(1.4f)));

#if defined(HEYOKA_WITH_SLEEF)
// NOTE: autovec with external scalar functions seems to work
// only since LLVM 16.
#if defined(HEYOKA_WITH_SLEEF) && LLVM_VERSION_MAJOR >= 16

const auto &tf = detail::get_target_features();

Expand All @@ -334,15 +336,11 @@ TEST_CASE("vfabi float")
REQUIRE(count == 5u);
}

#if LLVM_VERSION_MAJOR >= 16

// NOTE: LLVM16 is currently the version tested in the CI on arm64.
if (tf.aarch64) {
REQUIRE(count == 5u);
}

#endif

// NOTE: currently no auto-vectorization happens on ppc64 due apparently
// to the way the target machine is being set up by orc/lljit (it works
// fine with the opt tool). When this is resolved, we can test ppc64 too.
Expand Down
8 changes: 3 additions & 5 deletions test/atan.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -305,7 +305,9 @@ TEST_CASE("vfabi float")
REQUIRE(outs[2] == approximately(std::atan(.3f)));
REQUIRE(outs[3] == approximately(std::atan(.4f)));

#if defined(HEYOKA_WITH_SLEEF)
// NOTE: autovec with external scalar functions seems to work
// only since LLVM 16.
#if defined(HEYOKA_WITH_SLEEF) && LLVM_VERSION_MAJOR >= 16

const auto &tf = detail::get_target_features();

Expand All @@ -328,15 +330,11 @@ TEST_CASE("vfabi float")
REQUIRE(count == 5u);
}

#if LLVM_VERSION_MAJOR >= 16

// NOTE: LLVM16 is currently the version tested in the CI on arm64.
if (tf.aarch64) {
REQUIRE(count == 5u);
}

#endif

// NOTE: currently no auto-vectorization happens on ppc64 due apparently
// to the way the target machine is being set up by orc/lljit (it works
// fine with the opt tool). When this is resolved, we can test ppc64 too.
Expand Down

0 comments on commit ef89618

Please sign in to comment.