diff options
author | Matt Arsenault <Matthew.Arsenault@amd.com> | 2025-04-18 23:17:26 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2025-04-18 23:17:26 +0200 |
commit | 34e78093975b7c3be0fbd088429d8124702d8336 (patch) | |
tree | c5b1bfe051df15e5273cbcb83aba3fdf6f70f556 /llvm/unittests/Frontend/OpenMPIRBuilderTest.cpp | |
parent | 3ad353c91a0e759df5adb2d27c8b1ae214acab4f (diff) | |
download | llvm-34e78093975b7c3be0fbd088429d8124702d8336.zip llvm-34e78093975b7c3be0fbd088429d8124702d8336.tar.gz llvm-34e78093975b7c3be0fbd088429d8124702d8336.tar.bz2 |
unittests: Avoid using getNumUses (#136352)
Diffstat (limited to 'llvm/unittests/Frontend/OpenMPIRBuilderTest.cpp')
-rw-r--r-- | llvm/unittests/Frontend/OpenMPIRBuilderTest.cpp | 60 |
1 files changed, 30 insertions, 30 deletions
diff --git a/llvm/unittests/Frontend/OpenMPIRBuilderTest.cpp b/llvm/unittests/Frontend/OpenMPIRBuilderTest.cpp index 5dd0180..8300c31 100644 --- a/llvm/unittests/Frontend/OpenMPIRBuilderTest.cpp +++ b/llvm/unittests/Frontend/OpenMPIRBuilderTest.cpp @@ -325,7 +325,7 @@ static Value *findStoredValueInAggregateAt(LLVMContext &Ctx, Value *Aggregate, } EXPECT_NE(GEPAtIdx, nullptr); - EXPECT_EQ(GEPAtIdx->getNumUses(), 1U); + EXPECT_TRUE(GEPAtIdx->hasOneUse()); // Find the value stored to the aggregate. StoreInst *StoreToAgg = dyn_cast<StoreInst>(*GEPAtIdx->user_begin()); @@ -443,7 +443,7 @@ TEST_F(OpenMPIRBuilderTest, CreateCancel) { EXPECT_EQ(Cancel->getCalledFunction()->getName(), "__kmpc_cancel"); EXPECT_FALSE(Cancel->getCalledFunction()->doesNotAccessMemory()); EXPECT_FALSE(Cancel->getCalledFunction()->doesNotFreeMemory()); - EXPECT_EQ(Cancel->getNumUses(), 1U); + EXPECT_TRUE(Cancel->hasOneUse()); Instruction *CancelBBTI = Cancel->getParent()->getTerminator(); EXPECT_EQ(CancelBBTI->getNumSuccessors(), 2U); EXPECT_EQ(CancelBBTI->getSuccessor(0), NewIP.getBlock()); @@ -460,7 +460,7 @@ TEST_F(OpenMPIRBuilderTest, CreateCancel) { EXPECT_EQ(Barrier->getCalledFunction()->getName(), "__kmpc_cancel_barrier"); EXPECT_FALSE(Barrier->getCalledFunction()->doesNotAccessMemory()); EXPECT_FALSE(Barrier->getCalledFunction()->doesNotFreeMemory()); - EXPECT_EQ(Barrier->getNumUses(), 0U); + EXPECT_TRUE(Barrier->use_empty()); EXPECT_EQ(CancelBBTI->getSuccessor(1)->getTerminator()->getNumSuccessors(), 1U); EXPECT_EQ(CancelBBTI->getSuccessor(1)->getTerminator()->getSuccessor(0), CBB); @@ -516,7 +516,7 @@ TEST_F(OpenMPIRBuilderTest, CreateCancelIfCond) { EXPECT_EQ(Cancel->getCalledFunction()->getName(), "__kmpc_cancel"); EXPECT_FALSE(Cancel->getCalledFunction()->doesNotAccessMemory()); EXPECT_FALSE(Cancel->getCalledFunction()->doesNotFreeMemory()); - EXPECT_EQ(Cancel->getNumUses(), 1U); + EXPECT_TRUE(Cancel->hasOneUse()); Instruction *CancelBBTI = Cancel->getParent()->getTerminator(); EXPECT_EQ(CancelBBTI->getNumSuccessors(), 2U); EXPECT_EQ(CancelBBTI->getSuccessor(0)->size(), 1U); @@ -535,7 +535,7 @@ TEST_F(OpenMPIRBuilderTest, CreateCancelIfCond) { EXPECT_EQ(Barrier->getCalledFunction()->getName(), "__kmpc_cancel_barrier"); EXPECT_FALSE(Barrier->getCalledFunction()->doesNotAccessMemory()); EXPECT_FALSE(Barrier->getCalledFunction()->doesNotFreeMemory()); - EXPECT_EQ(Barrier->getNumUses(), 0U); + EXPECT_TRUE(Barrier->use_empty()); EXPECT_EQ(CancelBBTI->getSuccessor(1)->getTerminator()->getNumSuccessors(), 1U); EXPECT_EQ(CancelBBTI->getSuccessor(1)->getTerminator()->getSuccessor(0), CBB); @@ -586,7 +586,7 @@ TEST_F(OpenMPIRBuilderTest, CreateCancelBarrier) { EXPECT_EQ(Barrier->getCalledFunction()->getName(), "__kmpc_cancel_barrier"); EXPECT_FALSE(Barrier->getCalledFunction()->doesNotAccessMemory()); EXPECT_FALSE(Barrier->getCalledFunction()->doesNotFreeMemory()); - EXPECT_EQ(Barrier->getNumUses(), 1U); + EXPECT_TRUE(Barrier->hasOneUse()); Instruction *BarrierBBTI = Barrier->getParent()->getTerminator(); EXPECT_EQ(BarrierBBTI->getNumSuccessors(), 2U); EXPECT_EQ(BarrierBBTI->getSuccessor(0), NewIP.getBlock()); @@ -741,7 +741,7 @@ TEST_F(OpenMPIRBuilderTest, ParallelSimpleGPU) { EXPECT_EQ(OutlinedFn->getArg(2)->getType(), PointerType::get(M->getContext(), 0)); EXPECT_EQ(&OutlinedFn->getEntryBlock(), PrivAI->getParent()); - EXPECT_EQ(OutlinedFn->getNumUses(), 1U); + EXPECT_TRUE(OutlinedFn->hasOneUse()); User *Usr = OutlinedFn->user_back(); ASSERT_TRUE(isa<CallInst>(Usr)); CallInst *Parallel51CI = dyn_cast<CallInst>(Usr); @@ -849,7 +849,7 @@ TEST_F(OpenMPIRBuilderTest, ParallelSimple) { EXPECT_EQ(OutlinedFn->arg_size(), 3U); EXPECT_EQ(&OutlinedFn->getEntryBlock(), PrivAI->getParent()); - EXPECT_EQ(OutlinedFn->getNumUses(), 1U); + EXPECT_TRUE(OutlinedFn->hasOneUse()); User *Usr = OutlinedFn->user_back(); ASSERT_TRUE(isa<CallInst>(Usr)); CallInst *ForkCI = dyn_cast<CallInst>(Usr); @@ -952,7 +952,7 @@ TEST_F(OpenMPIRBuilderTest, ParallelNested) { EXPECT_TRUE(OutlinedFn.hasInternalLinkage()); EXPECT_EQ(OutlinedFn.arg_size(), 2U); - EXPECT_EQ(OutlinedFn.getNumUses(), 1U); + EXPECT_TRUE(OutlinedFn.hasOneUse()); User *Usr = OutlinedFn.user_back(); ASSERT_TRUE(isa<CallInst>(Usr)); CallInst *ForkCI = dyn_cast<CallInst>(Usr); @@ -1071,7 +1071,7 @@ TEST_F(OpenMPIRBuilderTest, ParallelNested2Inner) { NumAllocas += isa<AllocaInst>(I); EXPECT_EQ(NumAllocas, 1U); - EXPECT_EQ(OutlinedFn.getNumUses(), 1U); + EXPECT_TRUE(OutlinedFn.hasOneUse()); User *Usr = OutlinedFn.user_back(); ASSERT_TRUE(isa<CallInst>(Usr)); CallInst *ForkCI = dyn_cast<CallInst>(Usr); @@ -1178,7 +1178,7 @@ TEST_F(OpenMPIRBuilderTest, ParallelIfCond) { EXPECT_EQ(OutlinedFn->arg_size(), 3U); EXPECT_EQ(&OutlinedFn->getEntryBlock(), PrivAI->getParent()); - ASSERT_EQ(OutlinedFn->getNumUses(), 1U); + ASSERT_TRUE(OutlinedFn->hasOneUse()); CallInst *ForkCI = nullptr; for (User *Usr : OutlinedFn->users()) { @@ -1229,9 +1229,9 @@ TEST_F(OpenMPIRBuilderTest, ParallelCancelBarrier) { BFn = M->getFunction("__kmpc_barrier"); ASSERT_NE(CBFn, nullptr); ASSERT_EQ(BFn, nullptr); - ASSERT_EQ(CBFn->getNumUses(), 1U); + ASSERT_TRUE(CBFn->hasOneUse()); ASSERT_TRUE(isa<CallInst>(CBFn->user_back())); - ASSERT_EQ(CBFn->user_back()->getNumUses(), 1U); + ASSERT_TRUE(CBFn->user_back()->hasOneUse()); CheckedBarrier = cast<CallInst>(CBFn->user_back()); ASSERT_EXPECTED_INIT( @@ -1242,20 +1242,20 @@ TEST_F(OpenMPIRBuilderTest, ParallelCancelBarrier) { BFn = M->getFunction("__kmpc_barrier"); ASSERT_NE(CBFn, nullptr); ASSERT_NE(BFn, nullptr); - ASSERT_EQ(CBFn->getNumUses(), 1U); - ASSERT_EQ(BFn->getNumUses(), 1U); + ASSERT_TRUE(CBFn->hasOneUse()); + ASSERT_TRUE(BFn->hasOneUse()); ASSERT_TRUE(isa<CallInst>(BFn->user_back())); - ASSERT_EQ(BFn->user_back()->getNumUses(), 0U); + ASSERT_TRUE(BFn->user_back()->use_empty()); ASSERT_EXPECTED_INIT(OpenMPIRBuilder::InsertPointTy, BarrierIP3, OMPBuilder.createBarrier(Builder.saveIP(), OMPD_parallel, false, false)); Builder.restoreIP(BarrierIP3); - ASSERT_EQ(CBFn->getNumUses(), 2U); - ASSERT_EQ(BFn->getNumUses(), 1U); + ASSERT_TRUE(CBFn->hasNUses(2)); + ASSERT_TRUE(BFn->hasOneUse()); ASSERT_TRUE(CBFn->user_back() != CheckedBarrier); ASSERT_TRUE(isa<CallInst>(CBFn->user_back())); - ASSERT_EQ(CBFn->user_back()->getNumUses(), 0U); + ASSERT_TRUE(CBFn->user_back()->use_empty()); }; auto PrivCB = [&](InsertPointTy, InsertPointTy, Value &V, Value &, @@ -1289,7 +1289,7 @@ TEST_F(OpenMPIRBuilderTest, ParallelCancelBarrier) { EXPECT_EQ(NumBodiesGenerated, 1U); EXPECT_EQ(NumPrivatizedVars, 0U); EXPECT_EQ(NumFinalizationPoints, 2U); - EXPECT_EQ(FakeDestructor->getNumUses(), 2U); + EXPECT_TRUE(FakeDestructor->hasNUses(2)); Builder.restoreIP(AfterIP); Builder.CreateRetVoid(); @@ -3988,7 +3988,7 @@ TEST_F(OpenMPIRBuilderTest, OMPAtomicUpdate) { EXPECT_EQ(Phi->getIncomingBlock(0), EntryBB); EXPECT_EQ(Phi->getIncomingBlock(1), ContBB); - EXPECT_EQ(Sub->getNumUses(), 1U); + EXPECT_TRUE(Sub->hasOneUse()); StoreInst *St = dyn_cast<StoreInst>(Sub->user_back()); AllocaInst *UpdateTemp = dyn_cast<AllocaInst>(St->getPointerOperand()); @@ -4058,7 +4058,7 @@ TEST_F(OpenMPIRBuilderTest, OMPAtomicUpdateFloat) { EXPECT_EQ(Phi->getIncomingBlock(0), EntryBB); EXPECT_EQ(Phi->getIncomingBlock(1), ContBB); - EXPECT_EQ(Sub->getNumUses(), 1U); + EXPECT_TRUE(Sub->hasOneUse()); StoreInst *St = dyn_cast<StoreInst>(Sub->user_back()); AllocaInst *UpdateTemp = dyn_cast<AllocaInst>(St->getPointerOperand()); @@ -4127,7 +4127,7 @@ TEST_F(OpenMPIRBuilderTest, OMPAtomicUpdateIntr) { EXPECT_EQ(Phi->getIncomingBlock(0), EntryBB); EXPECT_EQ(Phi->getIncomingBlock(1), ContBB); - EXPECT_EQ(Sub->getNumUses(), 1U); + EXPECT_TRUE(Sub->hasOneUse()); StoreInst *St = dyn_cast<StoreInst>(Sub->user_back()); AllocaInst *UpdateTemp = dyn_cast<AllocaInst>(St->getPointerOperand()); @@ -4942,7 +4942,7 @@ TEST_F(OpenMPIRBuilderTest, CreateTeamsWithIfConditionAndNumTeams) { Value *ThreadLimitArg = PushNumTeamsCallInst->getArgOperand(4); // Get the boolean conversion of if expression - ASSERT_EQ(IfExpr->getNumUses(), 1U); + ASSERT_TRUE(IfExpr->hasOneUse()); User *IfExprInst = IfExpr->user_back(); ICmpInst *IfExprCmpInst = dyn_cast<ICmpInst>(IfExprInst); ASSERT_NE(IfExprCmpInst, nullptr); @@ -5272,7 +5272,7 @@ TEST_F(OpenMPIRBuilderTest, CreateReductions) { // Find the GEP instructions preceding stores to the local array. Value *FirstArrayElemPtr = nullptr; Value *SecondArrayElemPtr = nullptr; - EXPECT_EQ(LocalArray->getNumUses(), 3u); + EXPECT_TRUE(LocalArray->hasNUses(3)); ASSERT_TRUE( findGEPZeroOne(LocalArray, FirstArrayElemPtr, SecondArrayElemPtr)); @@ -5292,7 +5292,7 @@ TEST_F(OpenMPIRBuilderTest, CreateReductions) { // Check that the result of the runtime reduction call is used for further // dispatch. - ASSERT_EQ(SwitchArg->getNumUses(), 1u); + ASSERT_TRUE(SwitchArg->hasOneUse()); SwitchInst *Switch = dyn_cast<SwitchInst>(*SwitchArg->user_begin()); ASSERT_NE(Switch, nullptr); EXPECT_EQ(Switch->getNumSuccessors(), 3u); @@ -6296,7 +6296,7 @@ TEST_F(OpenMPIRBuilderTest, TargetRegion) { // Check num_teams and num_threads kernel arguments (use number 5 starting // from the end and counting the call to __tgt_target_kernel as the first use) Value *KernelArgs = Call->getArgOperand(Call->arg_size() - 1); - EXPECT_TRUE(KernelArgs->getNumUses() >= 4); + EXPECT_TRUE(KernelArgs->hasNUsesOrMore(4)); Value *NumTeamsGetElemPtr = *std::next(KernelArgs->user_begin(), 3); EXPECT_TRUE(isa<GetElementPtrInst>(NumTeamsGetElemPtr)); Value *NumTeamsStore = NumTeamsGetElemPtr->getUniqueUndroppableUser(); @@ -6622,7 +6622,7 @@ TEST_F(OpenMPIRBuilderTest, TargetRegionSPMD) { // Check the trip count kernel argument (use number 5 starting from the end // and counting the call to __tgt_target_kernel as the first use) Value *KernelArgs = Call->getArgOperand(Call->arg_size() - 1); - EXPECT_TRUE(KernelArgs->getNumUses() >= 6); + EXPECT_TRUE(KernelArgs->hasNUsesOrMore(6)); Value *TripCountGetElemPtr = *std::next(KernelArgs->user_begin(), 5); EXPECT_TRUE(isa<GetElementPtrInst>(TripCountGetElemPtr)); Value *TripCountStore = TripCountGetElemPtr->getUniqueUndroppableUser(); @@ -7024,7 +7024,7 @@ TEST_F(OpenMPIRBuilderTest, CreateTask) { // Verify that the data argument is used only once, and that too in the load // instruction that is then used for accessing shared data. Value *DataPtr = OutlinedFn->getArg(1); - EXPECT_EQ(DataPtr->getNumUses(), 1U); + EXPECT_TRUE(DataPtr->hasOneUse()); EXPECT_TRUE(isa<LoadInst>(DataPtr->uses().begin()->getUser())); Value *Data = DataPtr->uses().begin()->getUser(); EXPECT_TRUE(all_of(Data->uses(), [](Use &U) { @@ -7501,7 +7501,7 @@ TEST_F(OpenMPIRBuilderTest, CreateTaskgroupWithTasks) { Function *TaskAllocFn = OMPBuilder.getOrCreateRuntimeFunctionPtr(OMPRTL___kmpc_omp_task_alloc); - ASSERT_EQ(TaskAllocFn->getNumUses(), 2u); + ASSERT_TRUE(TaskAllocFn->hasNUses(2)); CallInst *FirstTaskAllocCall = dyn_cast_or_null<CallInst>(*TaskAllocFn->users().begin()); |