[mlir] Remove a redundant call to std::unique_ptr<T>::get (NFC) (#139409)

kazutakahirata · web-flow · commit fd98612accc2 · 2025-05-10T16:05:26.000-07:00
diff --git a/mlir/include/mlir/Analysis/DataFlowFramework.h b/mlir/include/mlir/Analysis/DataFlowFramework.h
@@ -707,7 +707,7 @@ template <typename AnalysisT, typename... Args>
 AnalysisT *DataFlowSolver::load(Args &&...args) {
   childAnalyses.emplace_back(new AnalysisT(*this, std::forward<Args>(args)...));
 #if LLVM_ENABLE_ABI_BREAKING_CHECKS
-  childAnalyses.back().get()->debugName = llvm::getTypeName<AnalysisT>();
+  childAnalyses.back()->debugName = llvm::getTypeName<AnalysisT>();
 #endif // LLVM_ENABLE_ABI_BREAKING_CHECKS
   return static_cast<AnalysisT *>(childAnalyses.back().get());
 }

Original file line number	Diff line number	Diff line change
`@@ -707,7 +707,7 @@ template <typename AnalysisT, typename... Args>`
`707`	`707`	`AnalysisT *DataFlowSolver::load(Args &&...args) {`
`708`	`708`	`childAnalyses.emplace_back(new AnalysisT(*this, std::forward<Args>(args)...));`
`709`	`709`	`#if LLVM_ENABLE_ABI_BREAKING_CHECKS`
`710`		`- childAnalyses.back().get()->debugName = llvm::getTypeName<AnalysisT>();`
	`710`	`+ childAnalyses.back()->debugName = llvm::getTypeName<AnalysisT>();`
`711`	`711`	`#endif // LLVM_ENABLE_ABI_BREAKING_CHECKS`
`712`	`712`	`return static_cast<AnalysisT *>(childAnalyses.back().get());`
`713`	`713`	`}`