[llvm] [Coroutines] Improve use of unique_ptr (PR #111870)
Tyler Nowicki via llvm-commits
llvm-commits at lists.llvm.org
Thu Oct 10 09:56:24 PDT 2024
https://github.com/TylerNowicki created https://github.com/llvm/llvm-project/pull/111870
* Replace usage of unique_ptr<>(new ...) -> make_unique<>();
>From cf17b511548709d8d98fe52ca6a854ca35b8f3cf Mon Sep 17 00:00:00 2001
From: tnowicki <tnowicki.nowicki at amd.com>
Date: Thu, 10 Oct 2024 12:45:31 -0400
Subject: [PATCH] [Coroutines] Improve use of unique_ptr
* Replace usage of unique_ptr<>(new ...) -> make_unique<>();
---
llvm/lib/Transforms/Coroutines/CoroSplit.cpp | 12 ++++--------
.../Transforms/Coroutines/ExtraRematTest.cpp | 2 +-
2 files changed, 5 insertions(+), 9 deletions(-)
diff --git a/llvm/lib/Transforms/Coroutines/CoroSplit.cpp b/llvm/lib/Transforms/Coroutines/CoroSplit.cpp
index 88ce331c8cfb64..0395ee62ae988b 100644
--- a/llvm/lib/Transforms/Coroutines/CoroSplit.cpp
+++ b/llvm/lib/Transforms/Coroutines/CoroSplit.cpp
@@ -2211,17 +2211,13 @@ CreateNewABI(Function &F, coro::Shape &S,
switch (S.ABI) {
case coro::ABI::Switch:
- return std::unique_ptr<coro::BaseABI>(
- new coro::SwitchABI(F, S, IsMatCallback));
+ return std::make_unique<coro::SwitchABI>(F, S, IsMatCallback);
case coro::ABI::Async:
- return std::unique_ptr<coro::BaseABI>(
- new coro::AsyncABI(F, S, IsMatCallback));
+ return std::make_unique<coro::AsyncABI>(F, S, IsMatCallback);
case coro::ABI::Retcon:
- return std::unique_ptr<coro::BaseABI>(
- new coro::AnyRetconABI(F, S, IsMatCallback));
+ return std::make_unique<coro::AnyRetconABI>(F, S, IsMatCallback);
case coro::ABI::RetconOnce:
- return std::unique_ptr<coro::BaseABI>(
- new coro::AnyRetconABI(F, S, IsMatCallback));
+ return std::make_unique<coro::AnyRetconABI>(F, S, IsMatCallback);
}
llvm_unreachable("Unknown ABI");
}
diff --git a/llvm/unittests/Transforms/Coroutines/ExtraRematTest.cpp b/llvm/unittests/Transforms/Coroutines/ExtraRematTest.cpp
index c3394fdaa940ba..68bf640334b5f2 100644
--- a/llvm/unittests/Transforms/Coroutines/ExtraRematTest.cpp
+++ b/llvm/unittests/Transforms/Coroutines/ExtraRematTest.cpp
@@ -247,7 +247,7 @@ TEST_F(ExtraRematTest, TestCoroRematWithCustomABI) {
ASSERT_TRUE(M);
CoroSplitPass::BaseABITy GenCustomABI = [](Function &F, coro::Shape &S) {
- return std::unique_ptr<coro::BaseABI>(new ExtraCustomABI(F, S));
+ return std::make_unique<ExtraCustomABI>(F, S);
};
CGSCCPassManager CGPM;
More information about the llvm-commits
mailing list