[llvm] [test] Disable makeAllowedFCmpRegion due to taking too long (PR #111056)

via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 3 13:21:59 PDT 2024


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-llvm-ir

Author: Arthur Eubanks (aeubanks)

<details>
<summary>Changes</summary>

This was added in #<!-- -->110082. It takes 32s in an optimized build of LLVM, and doesn't finish after multiple minutes in an -O0 build. It's iterating all possible fp values 3 levels deep. Disable until the test is fixed to not take so much time.

---
Full diff: https://github.com/llvm/llvm-project/pull/111056.diff


1 Files Affected:

- (modified) llvm/unittests/IR/ConstantFPRangeTest.cpp (+3) 


``````````diff
diff --git a/llvm/unittests/IR/ConstantFPRangeTest.cpp b/llvm/unittests/IR/ConstantFPRangeTest.cpp
index 17a08207fe1ba0..ae0ea8069fff5f 100644
--- a/llvm/unittests/IR/ConstantFPRangeTest.cpp
+++ b/llvm/unittests/IR/ConstantFPRangeTest.cpp
@@ -443,6 +443,9 @@ TEST_F(ConstantFPRangeTest, MismatchedSemantics) {
 #endif
 
 TEST_F(ConstantFPRangeTest, makeAllowedFCmpRegion) {
+  // TODO: reenable once test is fixed to run more quickly.
+  GTEST_SKIP();
+
   for (auto Pred : FCmpInst::predicates()) {
     EnumerateConstantFPRanges(
         [Pred](const ConstantFPRange &CR) {

``````````

</details>


https://github.com/llvm/llvm-project/pull/111056


More information about the llvm-commits mailing list