diff options
author | Craig Topper <craig.topper@gmail.com> | 2017-05-17 23:22:10 +0000 |
---|---|---|
committer | Craig Topper <craig.topper@gmail.com> | 2017-05-17 23:22:10 +0000 |
commit | 48187cffe2b55d99c655ad38aeca308d02303e95 (patch) | |
tree | ccf3f94fefa8ca5683dbf50093ce552e774f1bba | |
parent | a072d13e5467e7cb0a3d3612c525158b35f03723 (diff) | |
download | bcm5719-llvm-48187cffe2b55d99c655ad38aeca308d02303e95.tar.gz bcm5719-llvm-48187cffe2b55d99c655ad38aeca308d02303e95.zip |
[Statistics] Use Statistic::operator+= instead of adding and assigning separately.
I believe this technically fixes a multithreaded race condition in this code. But my primary concern was as part of looking at removing the ability to treat Statistics like a plain unsigned. There are many weird operations on Statistics in the codebase.
llvm-svn: 303314
-rw-r--r-- | llvm/lib/Transforms/Scalar/EarlyCSE.cpp | 2 | ||||
-rw-r--r-- | llvm/lib/Transforms/Scalar/LoadCombine.cpp | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/llvm/lib/Transforms/Scalar/EarlyCSE.cpp b/llvm/lib/Transforms/Scalar/EarlyCSE.cpp index d8f8a58a5fd..c4f450949e6 100644 --- a/llvm/lib/Transforms/Scalar/EarlyCSE.cpp +++ b/llvm/lib/Transforms/Scalar/EarlyCSE.cpp @@ -606,7 +606,7 @@ bool EarlyCSE::processNode(DomTreeNode *Node) { if (unsigned Count = replaceDominatedUsesWith( CondInst, TorF, DT, BasicBlockEdge(Pred, BB))) { Changed = true; - NumCSECVP = NumCSECVP + Count; + NumCSECVP += Count; } } } diff --git a/llvm/lib/Transforms/Scalar/LoadCombine.cpp b/llvm/lib/Transforms/Scalar/LoadCombine.cpp index 02215d3450c..494cbc61bc9 100644 --- a/llvm/lib/Transforms/Scalar/LoadCombine.cpp +++ b/llvm/lib/Transforms/Scalar/LoadCombine.cpp @@ -228,7 +228,7 @@ bool LoadCombine::combineLoads(SmallVectorImpl<LoadPOPPair> &Loads) { L.Load->replaceAllUsesWith(V); } - NumLoadsCombined = NumLoadsCombined + Loads.size(); + NumLoadsCombined += Loads.size(); return true; } |