github-actions[bot] wrote: <!--LLVM CODE FORMAT COMMENT: {clang-format}-->
:warning: C/C++ code formatter, clang-format found issues in your code. :warning: <details> <summary> You can test this locally with the following command: </summary> ``````````bash git-clang-format --diff HEAD~1 HEAD --extensions cpp,h,inc -- clang/lib/CodeGen/CGBlocks.cpp llvm/include/llvm/IR/Instruction.h llvm/include/llvm/Transforms/Utils/LockstepReverseIterator.h llvm/lib/Analysis/MemoryDependenceAnalysis.cpp llvm/lib/CodeGen/CodeGenPrepare.cpp llvm/lib/IR/Instruction.cpp llvm/lib/IR/Verifier.cpp llvm/lib/Target/AArch64/AArch64TargetTransformInfo.cpp llvm/lib/Target/X86/X86WinEHState.cpp llvm/lib/Transforms/Coroutines/CoroSplit.cpp llvm/lib/Transforms/IPO/Attributor.cpp llvm/lib/Transforms/IPO/AttributorAttributes.cpp llvm/lib/Transforms/IPO/GlobalOpt.cpp llvm/lib/Transforms/IPO/IROutliner.cpp llvm/lib/Transforms/IPO/OpenMPOpt.cpp llvm/lib/Transforms/InstCombine/InstCombineCalls.cpp llvm/lib/Transforms/InstCombine/InstructionCombining.cpp llvm/lib/Transforms/Instrumentation/AddressSanitizer.cpp llvm/lib/Transforms/Instrumentation/HWAddressSanitizer.cpp llvm/lib/Transforms/Instrumentation/ValueProfilePlugins.inc llvm/lib/Transforms/Scalar/LoopStrengthReduce.cpp llvm/lib/Transforms/Scalar/MemCpyOptimizer.cpp llvm/lib/Transforms/Utils/Local.cpp llvm/lib/Transforms/Utils/ScalarEvolutionExpander.cpp llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp llvm/unittests/Frontend/OpenMPIRBuilderTest.cpp llvm/unittests/FuzzMutate/RandomIRBuilderTest.cpp llvm/unittests/IR/InstructionsTest.cpp llvm/unittests/Transforms/Utils/LocalTest.cpp `````````` </details> <details> <summary> View the diff from clang-format here. </summary> ``````````diff diff --git a/llvm/lib/IR/Instruction.cpp b/llvm/lib/IR/Instruction.cpp index c6dca727e..dcede5b6c 100644 --- a/llvm/lib/IR/Instruction.cpp +++ b/llvm/lib/IR/Instruction.cpp @@ -1243,9 +1243,7 @@ Instruction::getPrevNonDebugInstruction(bool SkipPseudoOp) const { return nullptr; } -const DebugLoc &Instruction::getStableDebugLoc() const { - return getDebugLoc(); -} +const DebugLoc &Instruction::getStableDebugLoc() const { return getDebugLoc(); } bool Instruction::isAssociative() const { if (auto *II = dyn_cast<IntrinsicInst>(this)) diff --git a/llvm/lib/Transforms/Coroutines/CoroSplit.cpp b/llvm/lib/Transforms/Coroutines/CoroSplit.cpp index 1dc49e221..39e2ec529 100644 --- a/llvm/lib/Transforms/Coroutines/CoroSplit.cpp +++ b/llvm/lib/Transforms/Coroutines/CoroSplit.cpp @@ -814,8 +814,7 @@ static void updateScopeLine(Instruction *ActiveSuspend, return; } - BasicBlock::iterator Successor = - ActiveSuspend->getNextNode()->getIterator(); + BasicBlock::iterator Successor = ActiveSuspend->getNextNode()->getIterator(); // Corosplit splits the BB around ActiveSuspend, so the meaningful // instructions are not in the same BB. if (auto *Branch = dyn_cast_or_null<BranchInst>(Successor); diff --git a/llvm/lib/Transforms/IPO/AttributorAttributes.cpp b/llvm/lib/Transforms/IPO/AttributorAttributes.cpp index 4f5ba74e5..9c45e1e3b 100644 --- a/llvm/lib/Transforms/IPO/AttributorAttributes.cpp +++ b/llvm/lib/Transforms/IPO/AttributorAttributes.cpp @@ -1802,8 +1802,7 @@ ChangeStatus AAPointerInfoFloating::updateImpl(Attributor &A) { continue; return false; } - if (IsImpactedInRange(LoadI->getNextNode(), - BB->getTerminator())) + if (IsImpactedInRange(LoadI->getNextNode(), BB->getTerminator())) return false; if (IsImpactedInRange(&IntrBB->front(), &IntrI)) return false; diff --git a/llvm/lib/Transforms/IPO/IROutliner.cpp b/llvm/lib/Transforms/IPO/IROutliner.cpp index bdb16ae81..25a0804a0 100644 --- a/llvm/lib/Transforms/IPO/IROutliner.cpp +++ b/llvm/lib/Transforms/IPO/IROutliner.cpp @@ -267,8 +267,7 @@ void OutlinableRegion::splitCandidate() { // region is the same as the recorded instruction following the last // instruction. If they do not match, there could be problems in rewriting // the program after outlining, so we ignore it. - if (!BackInst->isTerminator() && - EndInst != BackInst->getNextNode()) + if (!BackInst->isTerminator() && EndInst != BackInst->getNextNode()) return; Instruction *StartInst = (*Candidate->begin()).Inst; diff --git a/llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp b/llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp index c68d9249b..b58bdae6b 100644 --- a/llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp +++ b/llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp @@ -16162,9 +16162,8 @@ void BoUpSLP::setInsertPointAfterBundle(const TreeEntry *E) { } else { // Set the insertion point after the last instruction in the bundle. Set the // debug location to Front. - Builder.SetInsertPoint( - LastInst->getParent(), - LastInst->getNextNode()->getIterator()); + Builder.SetInsertPoint(LastInst->getParent(), + LastInst->getNextNode()->getIterator()); } Builder.SetCurrentDebugLocation(Front->getDebugLoc()); } diff --git a/llvm/unittests/Frontend/OpenMPIRBuilderTest.cpp b/llvm/unittests/Frontend/OpenMPIRBuilderTest.cpp index 558af6900..2e2044397 100644 --- a/llvm/unittests/Frontend/OpenMPIRBuilderTest.cpp +++ b/llvm/unittests/Frontend/OpenMPIRBuilderTest.cpp @@ -7425,8 +7425,7 @@ TEST_F(OpenMPIRBuilderTest, CreateTaskgroup) { // Checking the general structure of the IR generated is same as expected. Instruction *GeneratedStoreInst = TaskgroupCall->getNextNode(); EXPECT_EQ(GeneratedStoreInst, InternalStoreInst); - Instruction *GeneratedLoad32 = - GeneratedStoreInst->getNextNode(); + Instruction *GeneratedLoad32 = GeneratedStoreInst->getNextNode(); EXPECT_EQ(GeneratedLoad32, InternalLoad32); Instruction *GeneratedLoad128 = GeneratedLoad32->getNextNode(); EXPECT_EQ(GeneratedLoad128, InternalLoad128); `````````` </details> https://github.com/llvm/llvm-project/pull/144383 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits