================
@@ -1193,11 +1206,26 @@ std::pair<bool, bool>
CodeGenPGO::getIsCounterPair(const Stmt *S) const {
}
void CodeGenPGO::emitCounterSetOrIncrement(CGBuilderTy &Builder, const Stmt *S,
+ bool UseSkipPath, bool UseBoth,
llvm::Value *StepV) {
- if (!RegionCounterMap || !Builder.GetInsertBlock())
+ if (!RegionCounterMap)
return;
- unsigned Counter = (*RegionCounterMap)[S].first;
+ unsigned Counter;
+ auto &TheMap = (*RegionCounterMap)[S];
+ auto IsCounter = TheMap.getIsCounterPair();
+ if (!UseSkipPath) {
+ if (!IsCounter.first)
+ return;
+ Counter = (TheMap.first & CounterPair::Mask);
+ } else {
+ if (!IsCounter.second)
+ return;
+ Counter = (TheMap.second & CounterPair::Mask);
+ }
+
+ if (!Builder.GetInsertBlock())
----------------
chapuni wrote:
It was the result of splitting the condition.
```
if (!RegionCounterMap || !Builder.GetInsertBlock())
return;
```
to
```
if (!RegionCounterMap)
return;
auto &TheMap = (*RegionCounterMap)[S]; // S should be allocated (as an initial
value)
...
if (!Builder.GetInsertBlock())
return;
```
https://github.com/llvm/llvm-project/pull/112730
_______________________________________________
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
