[llvm] 4e29d25 - Fix OpenMP deduplicateRuntimeCalls return status

[via llvm-commits]

Author: serge-sans-paille
Date: 2020-08-27T15:01:04+02:00
New Revision: 4e29d256694131cd8799b4aeccd1af42c3a2b7dd

URL: https://github.com/llvm/llvm-project/commit/4e29d256694131cd8799b4aeccd1af42c3a2b7dd
DIFF: https://github.com/llvm/llvm-project/commit/4e29d256694131cd8799b4aeccd1af42c3a2b7dd.diff

LOG: Fix OpenMP deduplicateRuntimeCalls return status

Differential Revision: https://reviews.llvm.org/D86705

Added: 
    

Modified: 
    llvm/lib/Transforms/IPO/OpenMPOpt.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Transforms/IPO/OpenMPOpt.cpp b/llvm/lib/Transforms/IPO/OpenMPOpt.cpp
index 39db2a36f56a..a63d9fdfdac1 100644
--- a/llvm/lib/Transforms/IPO/OpenMPOpt.cpp
+++ b/llvm/lib/Transforms/IPO/OpenMPOpt.cpp
@@ -657,8 +657,8 @@ struct OpenMPOpt {
 
     for (Function *F : SCC) {
       for (auto DeduplicableRuntimeCallID : DeduplicableRuntimeCallIDs)
-        deduplicateRuntimeCalls(*F,
-                                OMPInfoCache.RFIs[DeduplicableRuntimeCallID]);
+        Changed |= deduplicateRuntimeCalls(
+            *F, OMPInfoCache.RFIs[DeduplicableRuntimeCallID]);
 
       // __kmpc_global_thread_num is special as we can replace it with an
       // argument in enough cases to make it worth trying.