Skip to content

Commit

Permalink
[NFC] Fix a few whitespace inconsistencies.
Browse files Browse the repository at this point in the history
  • Loading branch information
paulwalker-arm committed Oct 20, 2022
1 parent 9cfe8ee commit ab8257c
Show file tree
Hide file tree
Showing 8 changed files with 10 additions and 13 deletions.
8 changes: 4 additions & 4 deletions clang/include/clang/Basic/DiagnosticSemaKinds.td
Expand Up @@ -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">>;
Expand Down
2 changes: 1 addition & 1 deletion llvm/include/llvm/IR/DIBuilder.h
Expand Up @@ -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.
Expand Down
2 changes: 2 additions & 0 deletions llvm/lib/Analysis/CostModel.cpp
Expand Up @@ -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
Expand Down Expand Up @@ -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
Expand Down
2 changes: 1 addition & 1 deletion llvm/lib/Analysis/VFABIDemangling.cpp
Expand Up @@ -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.
Expand All @@ -292,7 +293,6 @@ bool verifyAllVectorsHaveSameWidth(FunctionType *Signature) {
return (EC == VTy->getElementCount());
});
}

#endif // NDEBUG

// Extract the VectorizationFactor from a given function signature,
Expand Down
2 changes: 1 addition & 1 deletion llvm/lib/Analysis/ValueTracking.cpp
Expand Up @@ -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
Expand Down
1 change: 0 additions & 1 deletion llvm/lib/CodeGen/SelectionDAG/LegalizeIntegerTypes.cpp
Expand Up @@ -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);
Expand Down
5 changes: 0 additions & 5 deletions llvm/lib/Passes/PassBuilderPipelines.cpp
Expand Up @@ -204,7 +204,6 @@ PipelineTuningOptions::PipelineTuningOptions() {
}

namespace llvm {

extern cl::opt<unsigned> MaxDevirtIterations;
extern cl::opt<bool> EnableConstraintElimination;
extern cl::opt<bool> EnableFunctionSpecialization;
Expand All @@ -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
Expand Down
1 change: 1 addition & 0 deletions llvm/lib/Transforms/Vectorize/LoopVectorizationPlanner.h
Expand Up @@ -188,6 +188,7 @@ class VPBuilder {
struct VectorizationFactor {
/// Vector width with best cost.
ElementCount Width;

/// Cost of the loop with that width.
InstructionCost Cost;

Expand Down

0 comments on commit ab8257c

Please sign in to comment.