[llvm] 18be23f - [OPMIRBuilder] Fix typo in condition

Nikita Popov via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 27 03:24:10 PDT 2023


Author: Nikita Popov
Date: 2023-09-27T12:24:02+02:00
New Revision: 18be23f82a4c66ef6c838d3324979825acbcf49f

URL: https://github.com/llvm/llvm-project/commit/18be23f82a4c66ef6c838d3324979825acbcf49f
DIFF: https://github.com/llvm/llvm-project/commit/18be23f82a4c66ef6c838d3324979825acbcf49f.diff

LOG: [OPMIRBuilder] Fix typo in condition

Fix a condition I accidentally inverted in
296671f059198185bedc3eba80056ebf4bc7ad30.

Added: 
    

Modified: 
    llvm/lib/Frontend/OpenMP/OMPIRBuilder.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Frontend/OpenMP/OMPIRBuilder.cpp b/llvm/lib/Frontend/OpenMP/OMPIRBuilder.cpp
index fc15c565ca890f3..4eb91d8926d46a2 100644
--- a/llvm/lib/Frontend/OpenMP/OMPIRBuilder.cpp
+++ b/llvm/lib/Frontend/OpenMP/OMPIRBuilder.cpp
@@ -3523,7 +3523,7 @@ static int32_t computeHeuristicUnrollFactor(CanonicalLoopInfo *CLI) {
   UnrollCostEstimator UCE(L, TTI, EphValues, UP.BEInsns);
 
   // Loop is not unrollable if the loop contains certain instructions.
-  if (UCE.canUnroll() || UCE.Convergent) {
+  if (!UCE.canUnroll() || UCE.Convergent) {
     LLVM_DEBUG(dbgs() << "Loop not considered unrollable\n");
     return 1;
   }


        


More information about the llvm-commits mailing list