Author: Paul Walker Date: 2022-10-20T14:52:25Z New Revision: ab8257ca0e9389d39c081d7bb8bd4afceb2215cc
URL: https://github.com/llvm/llvm-project/commit/ab8257ca0e9389d39c081d7bb8bd4afceb2215cc DIFF: https://github.com/llvm/llvm-project/commit/ab8257ca0e9389d39c081d7bb8bd4afceb2215cc.diff LOG: [NFC] Fix a few whitespace inconsistencies. Added: Modified: clang/include/clang/Basic/DiagnosticSemaKinds.td llvm/include/llvm/IR/DIBuilder.h llvm/lib/Analysis/CostModel.cpp llvm/lib/Analysis/VFABIDemangling.cpp llvm/lib/Analysis/ValueTracking.cpp llvm/lib/CodeGen/SelectionDAG/LegalizeIntegerTypes.cpp llvm/lib/Passes/PassBuilderPipelines.cpp llvm/lib/Transforms/Vectorize/LoopVectorizationPlanner.h Removed: ################################################################################ diff --git a/clang/include/clang/Basic/DiagnosticSemaKinds.td b/clang/include/clang/Basic/DiagnosticSemaKinds.td index f8fe9555f8c02..1ef1f23e8c876 100644 --- a/clang/include/clang/Basic/DiagnosticSemaKinds.td +++ b/clang/include/clang/Basic/DiagnosticSemaKinds.td @@ -9960,10 +9960,10 @@ def err_argument_not_shifted_byte_or_xxff : Error< "argument should be an 8-bit value shifted by a multiple of 8 bits, or in the form 0x??FF">; def err_argument_not_contiguous_bit_field : Error< "argument %0 value should represent a contiguous bit field">; -def err_rotation_argument_to_cadd - : Error<"argument should be the value 90 or 270">; -def err_rotation_argument_to_cmla - : Error<"argument should be the value 0, 90, 180 or 270">; +def err_rotation_argument_to_cadd : Error< + "argument should be the value 90 or 270">; +def err_rotation_argument_to_cmla : Error< + "argument should be the value 0, 90, 180 or 270">; def warn_neon_vector_initializer_non_portable : Warning< "vector initializers are not compatible with NEON intrinsics in big endian " "mode">, InGroup<DiagGroup<"nonportable-vector-initialization">>; diff --git a/llvm/include/llvm/IR/DIBuilder.h b/llvm/include/llvm/IR/DIBuilder.h index 371a70fd1ee9c..dfb65ce341296 100644 --- a/llvm/include/llvm/IR/DIBuilder.h +++ b/llvm/include/llvm/IR/DIBuilder.h @@ -685,7 +685,7 @@ namespace llvm { DIGlobalVariable *createTempGlobalVariableFwdDecl( DIScope *Context, StringRef Name, StringRef LinkageName, DIFile *File, unsigned LineNo, DIType *Ty, bool IsLocalToUnit, MDNode *Decl = nullptr, - MDTuple *TemplateParams= nullptr, uint32_t AlignInBits = 0); + MDTuple *TemplateParams = nullptr, uint32_t AlignInBits = 0); /// Create a new descriptor for an auto variable. This is a local variable /// that is not a subprogram parameter. diff --git a/llvm/lib/Analysis/CostModel.cpp b/llvm/lib/Analysis/CostModel.cpp index c6a68522993b8..1782b399e7fd0 100644 --- a/llvm/lib/Analysis/CostModel.cpp +++ b/llvm/lib/Analysis/CostModel.cpp @@ -109,6 +109,7 @@ void CostModelAnalysis::print(raw_ostream &OS, const Module*) const { else { Cost = TTI->getInstructionCost(&Inst, CostKind); } + if (auto CostVal = Cost.getValue()) OS << "Cost Model: Found an estimated cost of " << *CostVal; else @@ -137,6 +138,7 @@ PreservedAnalyses CostModelPrinterPass::run(Function &F, else { Cost = TTI.getInstructionCost(&Inst, CostKind); } + if (auto CostVal = Cost.getValue()) OS << "Cost Model: Found an estimated cost of " << *CostVal; else diff --git a/llvm/lib/Analysis/VFABIDemangling.cpp b/llvm/lib/Analysis/VFABIDemangling.cpp index fa483d00826a4..aa38c39ae71a1 100644 --- a/llvm/lib/Analysis/VFABIDemangling.cpp +++ b/llvm/lib/Analysis/VFABIDemangling.cpp @@ -272,6 +272,7 @@ ParseRet tryParseAlign(StringRef &ParseString, Align &Alignment) { return ParseRet::None; } + #ifndef NDEBUG // Verify the assumtion that all vectors in the signature of a vector // function have the same number of elements. @@ -292,7 +293,6 @@ bool verifyAllVectorsHaveSameWidth(FunctionType *Signature) { return (EC == VTy->getElementCount()); }); } - #endif // NDEBUG // Extract the VectorizationFactor from a given function signature, diff --git a/llvm/lib/Analysis/ValueTracking.cpp b/llvm/lib/Analysis/ValueTracking.cpp index 685910c94b5f0..563929a9771c7 100644 --- a/llvm/lib/Analysis/ValueTracking.cpp +++ b/llvm/lib/Analysis/ValueTracking.cpp @@ -2297,7 +2297,7 @@ static bool isGEPKnownNonNull(const GEPOperator *GEP, unsigned Depth, } // If we have a zero-sized type, the index doesn't matter. Keep looping. - if (Q.DL.getTypeAllocSize(GTI.getIndexedType()).getKnownMinSize() == 0) + if (Q.DL.getTypeAllocSize(GTI.getIndexedType()).isZero()) continue; // Fast path the constant operand case both for efficiency and so we don't diff --git a/llvm/lib/CodeGen/SelectionDAG/LegalizeIntegerTypes.cpp b/llvm/lib/CodeGen/SelectionDAG/LegalizeIntegerTypes.cpp index 96d120d513ade..7b8e0423041f6 100644 --- a/llvm/lib/CodeGen/SelectionDAG/LegalizeIntegerTypes.cpp +++ b/llvm/lib/CodeGen/SelectionDAG/LegalizeIntegerTypes.cpp @@ -5243,7 +5243,6 @@ SDValue DAGTypeLegalizer::PromoteIntRes_VECTOR_SHUFFLE(SDNode *N) { return DAG.getVectorShuffle(OutVT, dl, V0, V1, NewMask); } - SDValue DAGTypeLegalizer::PromoteIntRes_BUILD_VECTOR(SDNode *N) { EVT OutVT = N->getValueType(0); EVT NOutVT = TLI.getTypeToTransformTo(*DAG.getContext(), OutVT); diff --git a/llvm/lib/Passes/PassBuilderPipelines.cpp b/llvm/lib/Passes/PassBuilderPipelines.cpp index 6e39df34193b1..4f1ef22775db8 100644 --- a/llvm/lib/Passes/PassBuilderPipelines.cpp +++ b/llvm/lib/Passes/PassBuilderPipelines.cpp @@ -204,7 +204,6 @@ PipelineTuningOptions::PipelineTuningOptions() { } namespace llvm { - extern cl::opt<unsigned> MaxDevirtIterations; extern cl::opt<bool> EnableConstraintElimination; extern cl::opt<bool> EnableFunctionSpecialization; @@ -221,14 +220,10 @@ extern cl::opt<bool> EnableDFAJumpThreading; extern cl::opt<bool> RunNewGVN; extern cl::opt<bool> RunPartialInlining; extern cl::opt<bool> ExtraVectorizerPasses; - extern cl::opt<bool> FlattenedProfileUsed; - extern cl::opt<AttributorRunOption> AttributorRun; extern cl::opt<bool> EnableKnowledgeRetention; - extern cl::opt<bool> EnableMatrix; - extern cl::opt<bool> DisablePreInliner; extern cl::opt<int> PreInlineThreshold; } // namespace llvm diff --git a/llvm/lib/Transforms/Vectorize/LoopVectorizationPlanner.h b/llvm/lib/Transforms/Vectorize/LoopVectorizationPlanner.h index 2e9a9fe0640ea..7c7307905e8d4 100644 --- a/llvm/lib/Transforms/Vectorize/LoopVectorizationPlanner.h +++ b/llvm/lib/Transforms/Vectorize/LoopVectorizationPlanner.h @@ -188,6 +188,7 @@ class VPBuilder { struct VectorizationFactor { /// Vector width with best cost. ElementCount Width; + /// Cost of the loop with that width. InstructionCost Cost; _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
