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 cpp,c,h --
clang/test/CodeGen/splatvector-sizeless-aarch64.c
clang/test/CodeGen/splatvector-sizeless-riscv.c
clang/test/CodeGen/splatvector-template.cpp clang/test/CodeGen/splatvector.c
clang/test/Sema/splatvector.c clang/include/clang/AST/ComputeDependence.h
clang/include/clang/AST/Expr.h clang/include/clang/AST/RecursiveASTVisitor.h
clang/include/clang/AST/Stmt.h clang/include/clang/AST/TextNodeDumper.h
clang/include/clang/ASTMatchers/ASTMatchers.h clang/include/clang/Sema/Sema.h
clang/include/clang/Serialization/ASTBitCodes.h clang/lib/AST/ASTImporter.cpp
clang/lib/AST/ComputeDependence.cpp clang/lib/AST/Expr.cpp
clang/lib/AST/ExprClassification.cpp clang/lib/AST/ExprConstant.cpp
clang/lib/AST/ItaniumMangle.cpp clang/lib/AST/StmtPrinter.cpp
clang/lib/AST/StmtProfile.cpp clang/lib/AST/TextNodeDumper.cpp
clang/lib/ASTMatchers/ASTMatchersInternal.cpp
clang/lib/ASTMatchers/Dynamic/Registry.cpp clang/lib/CodeGen/CGExprScalar.cpp
clang/lib/Parse/ParseExpr.cpp clang/lib/Sema/SemaChecking.cpp
clang/lib/Sema/SemaExceptionSpec.cpp clang/lib/Sema/SemaExpr.cpp
clang/lib/Sema/TreeTransform.h clang/lib/Serialization/ASTReaderStmt.cpp
clang/lib/Serialization/ASTWriterStmt.cpp
clang/lib/StaticAnalyzer/Core/ExprEngine.cpp
clang/test/AST/ast-dump-fpfeatures.cpp clang/test/CodeGen/pragma-fenv_access.c
clang/test/PCH/exprs.h clang/test/Preprocessor/feature_tests.c
clang/test/Preprocessor/feature_tests.cpp clang/test/Sema/vector-bool-assign.c
clang/test/Sema/vector-bool-assign.cpp clang/tools/libclang/CXCursor.cpp
clang/unittests/AST/ASTImporterTest.cpp
clang/unittests/ASTMatchers/ASTMatchersNodeTest.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/AST/Expr.h b/clang/include/clang/AST/Expr.h
index 669247d15..fb0acf855 100644
--- a/clang/include/clang/AST/Expr.h
+++ b/clang/include/clang/AST/Expr.h
@@ -4781,9 +4781,7 @@ public:
}
/// Is FPFeatures in Trailing Storage?
- bool hasStoredFPFeatures() const {
- return SplatVectorExprBits.HasFPFeatures;
- }
+ bool hasStoredFPFeatures() const { return SplatVectorExprBits.HasFPFeatures;
}
/// Get FPFeatures from trailing storage.
FPOptionsOverride getStoredFPFeatures() const {
@@ -4816,12 +4814,8 @@ public:
Expr *getSrcExpr() const { return cast<Expr>(SrcExpr); }
/// getTypeSourceInfo - Return the destination type.
- TypeSourceInfo *getTypeSourceInfo() const {
- return TInfo;
- }
- void setTypeSourceInfo(TypeSourceInfo *ti) {
- TInfo = ti;
- }
+ TypeSourceInfo *getTypeSourceInfo() const { return TInfo; }
+ void setTypeSourceInfo(TypeSourceInfo *ti) { TInfo = ti; }
/// getBuiltinLoc - Return the location of the __builtin_splatvector token.
SourceLocation getBuiltinLoc() const { return BuiltinLoc; }
@@ -4837,7 +4831,7 @@ public:
}
// Iterators
- child_range children() { return child_range(&SrcExpr, &SrcExpr+1); }
+ child_range children() { return child_range(&SrcExpr, &SrcExpr + 1); }
const_child_range children() const {
return const_child_range(&SrcExpr, &SrcExpr + 1);
}
diff --git a/clang/lib/AST/ASTImporter.cpp b/clang/lib/AST/ASTImporter.cpp
index 9c28fdb3a..bc640c97d 100644
--- a/clang/lib/AST/ASTImporter.cpp
+++ b/clang/lib/AST/ASTImporter.cpp
@@ -7815,10 +7815,10 @@ ExpectedStmt
ASTNodeImporter::VisitSplatVectorExpr(SplatVectorExpr *E) {
if (Err)
return std::move(Err);
- return SplatVectorExpr::Create(
- Importer.getToContext(), ToSrcExpr, ToTSI, ToType, E->getValueKind(),
- E->getObjectKind(), ToBuiltinLoc, ToRParenLoc,
- E->getStoredFPFeaturesOrDefault());
+ return SplatVectorExpr::Create(Importer.getToContext(), ToSrcExpr, ToTSI,
+ ToType, E->getValueKind(), E->getObjectKind(),
+ ToBuiltinLoc, ToRParenLoc,
+ E->getStoredFPFeaturesOrDefault());
}
ExpectedStmt ASTNodeImporter::VisitConvertVectorExpr(ConvertVectorExpr *E) {
diff --git a/clang/lib/AST/Expr.cpp b/clang/lib/AST/Expr.cpp
index 4644e3a86..cd92d9290 100644
--- a/clang/lib/AST/Expr.cpp
+++ b/clang/lib/AST/Expr.cpp
@@ -5694,10 +5694,12 @@ SplatVectorExpr *SplatVectorExpr::CreateEmpty(const
ASTContext &C,
return new (Mem) SplatVectorExpr(hasFPFeatures, EmptyShell());
}
-SplatVectorExpr *SplatVectorExpr::Create(
- const ASTContext &C, Expr *SrcExpr, TypeSourceInfo *TI, QualType DstType,
- ExprValueKind VK, ExprObjectKind OK, SourceLocation BuiltinLoc,
- SourceLocation RParenLoc, FPOptionsOverride FPFeatures) {
+SplatVectorExpr *SplatVectorExpr::Create(const ASTContext &C, Expr *SrcExpr,
+ TypeSourceInfo *TI, QualType DstType,
+ ExprValueKind VK, ExprObjectKind OK,
+ SourceLocation BuiltinLoc,
+ SourceLocation RParenLoc,
+ FPOptionsOverride FPFeatures) {
bool HasFPFeatures = FPFeatures.requiresTrailingStorage();
unsigned Size = totalSizeToAlloc<FPOptionsOverride>(HasFPFeatures);
void *Mem = C.Allocate(Size, alignof(SplatVectorExpr));
diff --git a/clang/lib/CodeGen/CGExprScalar.cpp
b/clang/lib/CodeGen/CGExprScalar.cpp
index cde834a71..16c2f91c1 100644
--- a/clang/lib/CodeGen/CGExprScalar.cpp
+++ b/clang/lib/CodeGen/CGExprScalar.cpp
@@ -2080,10 +2080,9 @@ static QualType getRawVectorElementType(ASTContext
&Context,
Value *ScalarExprEmitter::VisitSplatVectorExpr(SplatVectorExpr *E) {
CodeGenFunction::CGFPOptionsRAII FPOptions(CGF, E);
- QualType SrcType = E->getSrcExpr()->getType(),
- DstType = E->getType();
+ QualType SrcType = E->getSrcExpr()->getType(), DstType = E->getType();
- Value *Src = CGF.EmitScalarExpr(E->getSrcExpr());
+ Value *Src = CGF.EmitScalarExpr(E->getSrcExpr());
SrcType = CGF.getContext().getCanonicalType(SrcType);
DstType = CGF.getContext().getCanonicalType(DstType);
@@ -2099,14 +2098,15 @@ Value
*ScalarExprEmitter::VisitSplatVectorExpr(SplatVectorExpr *E) {
QualType DstEltType = getRawVectorElementType(CGF.getContext(), DstType);
llvm::Type *DstEltTy = ConvertType(DstEltType);
- Value *Splat =
Builder.CreateVectorSplat(cast<llvm::VectorType>(DstTy)->getElementCount(),
Src, "splatvector");
+ Value *Splat = Builder.CreateVectorSplat(
+ cast<llvm::VectorType>(DstTy)->getElementCount(), Src, "splatvector");
llvm::Type *SplatTy = Splat->getType();
if (SplatTy == DstTy)
return Splat;
if (DstEltType->isBooleanType()) {
- assert((SrcTy->isFloatingPointTy() ||
- isa<llvm::IntegerType>(SrcTy)) && "Unknown boolean conversion");
+ assert((SrcTy->isFloatingPointTy() || isa<llvm::IntegerType>(SrcTy)) &&
+ "Unknown boolean conversion");
llvm::Value *Zero = llvm::Constant::getNullValue(SplatTy);
if (SrcTy->isFloatingPointTy())
diff --git a/clang/lib/Sema/SemaChecking.cpp b/clang/lib/Sema/SemaChecking.cpp
index 28f8050bf..6270a7aab 100644
--- a/clang/lib/Sema/SemaChecking.cpp
+++ b/clang/lib/Sema/SemaChecking.cpp
@@ -6641,8 +6641,7 @@ ExprResult Sema::SplatVectorExpr(Expr *E, TypeSourceInfo
*TInfo,
QualType SrcTy = E->getType();
if (SrcTy->isVectorType() || SrcTy->isSizelessVectorType())
- return ExprError(Diag(BuiltinLoc,
- diag::err_splatvector_non_scalar)
+ return ExprError(Diag(BuiltinLoc, diag::err_splatvector_non_scalar)
<< E->getSourceRange());
if (!DstTy->isVectorType() && !DstTy->isSizelessVectorType() &&
!DstTy->isDependentType())
diff --git a/clang/lib/Sema/TreeTransform.h b/clang/lib/Sema/TreeTransform.h
index 76d541cd6..ff35617d7 100644
--- a/clang/lib/Sema/TreeTransform.h
+++ b/clang/lib/Sema/TreeTransform.h
@@ -4060,8 +4060,8 @@ public:
}
/// Build a new splat vector expression.
- ExprResult RebuildSplatVectorExpr(SourceLocation BuiltinLoc,
- Expr *SrcExpr, TypeSourceInfo *DstTInfo,
+ ExprResult RebuildSplatVectorExpr(SourceLocation BuiltinLoc, Expr *SrcExpr,
+ TypeSourceInfo *DstTInfo,
SourceLocation RParenLoc) {
return SemaRef.SplatVectorExpr(SrcExpr, DstTInfo, BuiltinLoc, RParenLoc);
}
@@ -17666,7 +17666,7 @@
TreeTransform<Derived>::TransformShuffleVectorExpr(ShuffleVectorExpr *E) {
E->getRParenLoc());
}
-template<typename Derived>
+template <typename Derived>
ExprResult
TreeTransform<Derived>::TransformSplatVectorExpr(SplatVectorExpr *E) {
ExprResult SrcExpr = getDerived().TransformExpr(E->getSrcExpr());
@@ -17677,14 +17677,12 @@
TreeTransform<Derived>::TransformSplatVectorExpr(SplatVectorExpr *E) {
if (!Type)
return ExprError();
- if (!getDerived().AlwaysRebuild() &&
- Type == E->getTypeSourceInfo() &&
+ if (!getDerived().AlwaysRebuild() && Type == E->getTypeSourceInfo() &&
SrcExpr.get() == E->getSrcExpr())
return E;
- return getDerived().RebuildSplatVectorExpr(E->getBuiltinLoc(),
- SrcExpr.get(), Type,
- E->getRParenLoc());
+ return getDerived().RebuildSplatVectorExpr(E->getBuiltinLoc(), SrcExpr.get(),
+ Type, E->getRParenLoc());
}
template<typename Derived>
diff --git a/clang/unittests/AST/ASTImporterTest.cpp
b/clang/unittests/AST/ASTImporterTest.cpp
index e1be4a5d7..20251199f 100644
--- a/clang/unittests/AST/ASTImporterTest.cpp
+++ b/clang/unittests/AST/ASTImporterTest.cpp
@@ -468,8 +468,7 @@ TEST_P(ASTImporterOptionSpecificTestBase,
ImportSplatVectorExpr) {
"void declToImport() { (void)__builtin_splatvector(f, v4double); }",
Lang_CXX03, "", Lang_CXX03);
- auto ToResults =
- match(splatVectorExpr().bind("splat"), To->getASTContext());
+ auto ToResults = match(splatVectorExpr().bind("splat"), To->getASTContext());
auto FromResults =
match(splatVectorExpr().bind("splat"), From->getASTContext());
``````````
</details>
https://github.com/llvm/llvm-project/pull/212758
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
