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 origin/main HEAD --extensions h,cpp --
clang/include/clang/Analysis/Analyses/LifetimeSafety/Facts.h
clang/include/clang/Analysis/Analyses/LifetimeSafety/FactsGenerator.h
clang/include/clang/Analysis/Analyses/LifetimeSafety/Loans.h
clang/lib/Analysis/LifetimeSafety/Checker.cpp
clang/lib/Analysis/LifetimeSafety/Facts.cpp
clang/lib/Analysis/LifetimeSafety/FactsGenerator.cpp
clang/lib/Analysis/LifetimeSafety/LoanPropagation.cpp
clang/lib/Analysis/LifetimeSafety/Loans.cpp
clang/lib/Analysis/LifetimeSafety/MovedLoans.cpp
clang/test/Sema/warn-lifetime-safety-invalidations.cpp --diff_from_common_commit
``````````
:warning:
The reproduction instructions above might return results for more than one PR
in a stack if you are using a stacked PR workflow. You can limit the results by
changing `origin/main` to the base branch/commit you want to compare against.
:warning:
</details>
<details>
<summary>
View the diff from clang-format here.
</summary>
``````````diff
diff --git a/clang/include/clang/Analysis/Analyses/LifetimeSafety/Loans.h
b/clang/include/clang/Analysis/Analyses/LifetimeSafety/Loans.h
index 49d2ddccf..e8546e656 100644
--- a/clang/include/clang/Analysis/Analyses/LifetimeSafety/Loans.h
+++ b/clang/include/clang/Analysis/Analyses/LifetimeSafety/Loans.h
@@ -38,7 +38,9 @@ public:
static PathElement getField(const FieldDecl *FD) {
return PathElement(Kind::Field, FD);
}
- static PathElement getInterior() { return PathElement(Kind::Interior,
nullptr); }
+ static PathElement getInterior() {
+ return PathElement(Kind::Interior, nullptr);
+ }
bool isField() const { return K == Kind::Field; }
bool isInterior() const { return K == Kind::Interior; }
diff --git a/clang/lib/Analysis/LifetimeSafety/Checker.cpp
b/clang/lib/Analysis/LifetimeSafety/Checker.cpp
index dc78c6497..a79a3b787 100644
--- a/clang/lib/Analysis/LifetimeSafety/Checker.cpp
+++ b/clang/lib/Analysis/LifetimeSafety/Checker.cpp
@@ -157,7 +157,8 @@ public:
for (LoanID HeldLoanID : HeldLoans) {
const Loan *HeldLoan = FactMgr.getLoanMgr().getLoan(HeldLoanID);
- if
(ExpiredLoan->getAccessPath().isPrefixOf(HeldLoan->getAccessPath())) {
+ if (ExpiredLoan->getAccessPath().isPrefixOf(
+ HeldLoan->getAccessPath())) {
// HeldLoan is expired because its base or itself is expired.
const Expr *MovedExpr = nullptr;
if (auto *ME = MovedLoans.getMovedLoans(EF).lookup(HeldLoanID))
diff --git a/clang/lib/Analysis/LifetimeSafety/FactsGenerator.cpp
b/clang/lib/Analysis/LifetimeSafety/FactsGenerator.cpp
index 61c1f2b40..d9a085c4d 100644
--- a/clang/lib/Analysis/LifetimeSafety/FactsGenerator.cpp
+++ b/clang/lib/Analysis/LifetimeSafety/FactsGenerator.cpp
@@ -79,7 +79,7 @@ static const Loan *createLoan(FactManager &FactMgr, const
DeclRefExpr *DRE) {
/// \param MTE The MaterializeTemporaryExpr that represents the temporary
/// binding. \return The new Loan.
static const Loan *createLoan(FactManager &FactMgr,
- const MaterializeTemporaryExpr *MTE) {
+ const MaterializeTemporaryExpr *MTE) {
AccessPath Path(MTE);
return FactMgr.getLoanMgr().createLoan(Path, MTE);
}
diff --git a/clang/lib/Analysis/LifetimeSafety/LoanPropagation.cpp
b/clang/lib/Analysis/LifetimeSafety/LoanPropagation.cpp
index e0bd544e3..f840983dc 100644
--- a/clang/lib/Analysis/LifetimeSafety/LoanPropagation.cpp
+++ b/clang/lib/Analysis/LifetimeSafety/LoanPropagation.cpp
@@ -181,9 +181,8 @@ public:
if (auto Element = F.getPathElement()) {
FlowLoans = LoanSetFactory.getEmptySet();
for (LoanID LID : SrcLoans) {
- Loan *ExtendedLoan =
- FactMgr.getLoanMgr().getOrCreateExtendedLoan(LID, *Element,
- nullptr);
+ Loan *ExtendedLoan = FactMgr.getLoanMgr().getOrCreateExtendedLoan(
+ LID, *Element, nullptr);
FlowLoans = LoanSetFactory.add(FlowLoans, ExtendedLoan->getID());
}
}
diff --git a/clang/lib/Analysis/LifetimeSafety/Loans.cpp
b/clang/lib/Analysis/LifetimeSafety/Loans.cpp
index 9c66b3d28..a429d1b99 100644
--- a/clang/lib/Analysis/LifetimeSafety/Loans.cpp
+++ b/clang/lib/Analysis/LifetimeSafety/Loans.cpp
@@ -38,7 +38,8 @@ LoanManager::getOrCreatePlaceholderBase(const ParmVarDecl
*PVD) {
llvm::FoldingSetNodeID ID;
ID.AddPointer(PVD);
void *InsertPos = nullptr;
- if (PlaceholderBase *Existing = PlaceholderBases.FindNodeOrInsertPos(ID,
InsertPos))
+ if (PlaceholderBase *Existing =
+ PlaceholderBases.FindNodeOrInsertPos(ID, InsertPos))
return Existing;
void *Mem = LoanAllocator.Allocate<PlaceholderBase>();
@@ -52,7 +53,8 @@ LoanManager::getOrCreatePlaceholderBase(const CXXMethodDecl
*MD) {
llvm::FoldingSetNodeID ID;
ID.AddPointer(MD);
void *InsertPos = nullptr;
- if (PlaceholderBase *Existing = PlaceholderBases.FindNodeOrInsertPos(ID,
InsertPos))
+ if (PlaceholderBase *Existing =
+ PlaceholderBases.FindNodeOrInsertPos(ID, InsertPos))
return Existing;
void *Mem = LoanAllocator.Allocate<PlaceholderBase>();
``````````
</details>
https://github.com/llvm/llvm-project/pull/180369
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
