[Conversion] Avoid repeated hash lookups (NFC) (#111637)

kazutakahirata · web-flow · commit 01a0e85ab718 · 2024-10-09T06:46:07.000-07:00
diff --git a/mlir/lib/Conversion/SCFToGPU/SCFToGPU.cpp b/mlir/lib/Conversion/SCFToGPU/SCFToGPU.cpp
@@ -509,12 +509,11 @@ static LogicalResult processParallelLoop(
                   ensureLaunchIndependent(cloningMap.lookupOrDefault(step))});
           // todo(herhut,ravishankarm): Update the behavior of setMappingAttr
           // when this condition is relaxed.
-          if (bounds.contains(processor)) {
+          if (!bounds.try_emplace(processor, launchBound).second) {
             return rewriter.notifyMatchFailure(
                 parallelOp, "cannot redefine the bound for processor " +
                                 Twine(static_cast<int64_t>(processor)));
           }
-          bounds[processor] = launchBound;
         }
         if (!boundIsPrecise) {
           // We are using an approximation, create a surrounding conditional.

Original file line number	Diff line number	Diff line change
`@@ -509,12 +509,11 @@ static LogicalResult processParallelLoop(`
`509`	`509`	`ensureLaunchIndependent(cloningMap.lookupOrDefault(step))});`
`510`	`510`	`// todo(herhut,ravishankarm): Update the behavior of setMappingAttr`
`511`	`511`	`// when this condition is relaxed.`
`512`		`- if (bounds.contains(processor)) {`
	`512`	`+ if (!bounds.try_emplace(processor, launchBound).second) {`
`513`	`513`	`return rewriter.notifyMatchFailure(`
`514`	`514`	`parallelOp, "cannot redefine the bound for processor " +`
`515`	`515`	`Twine(static_cast<int64_t>(processor)));`
`516`	`516`	`}`
`517`		`- bounds[processor] = launchBound;`
`518`	`517`	`}`
`519`	`518`	`if (!boundIsPrecise) {`
`520`	`519`	`// We are using an approximation, create a surrounding conditional.`