EISW-173755 Fix clang-format failures as part of Check code formatting/code_formatter (pull_request) CI check

Author: sramasit

llvm/include/llvm/ADT/FunctionExtras.h

@@ -111,8 +111,12 @@ template <typename ReturnT, typename... ParamTs> class UniqueFunctionBase {
   // This specialization ensures that 'AdjustedParam<V<T>&>' or
   // 'AdjustedParam<V<T>&&>' does not trigger a compile-time error when 'T' is
   // an incomplete type and V a templated type.
-  template <typename T> struct AdjustedParamTBase<T &> { using type = T &; };
-  template <typename T> struct AdjustedParamTBase<T &&> { using type = T &; };
+  template <typename T> struct AdjustedParamTBase<T &> {
+    using type = T &;
+  };
+  template <typename T> struct AdjustedParamTBase<T &&> {
+    using type = T &;
+  };
 
   template <typename T>
   using AdjustedParamT = typename AdjustedParamTBase<T>::type;

llvm/lib/DebugInfo/DWARF/DWARFVerifier.cpp

@@ -154,11 +154,13 @@ bool DWARFVerifier::verifyUnitHeader(const DWARFDataExtractor DebugInfoData,
   if (Version >= 5) {
     UnitType = DebugInfoData.getU8(Offset);
     AddrSize = DebugInfoData.getU8(Offset);
-    AbbrOffset = isUnitDWARF64 ? DebugInfoData.getU64(Offset) : DebugInfoData.getU32(Offset);
+    AbbrOffset = isUnitDWARF64 ? DebugInfoData.getU64(Offset)
+                               : DebugInfoData.getU32(Offset);
     ValidType = dwarf::isUnitType(UnitType);
   } else {
     UnitType = 0;
-    AbbrOffset = isUnitDWARF64 ? DebugInfoData.getU64(Offset) : DebugInfoData.getU32(Offset);
+    AbbrOffset = isUnitDWARF64 ? DebugInfoData.getU64(Offset)
+                               : DebugInfoData.getU32(Offset);
     AddrSize = DebugInfoData.getU8(Offset);
   }
 
@@ -420,7 +422,7 @@ unsigned DWARFVerifier::verifyUnits(const DWARFUnitVector &Units) {
   unsigned Index = 1;
   for (const auto &Unit : Units) {
     OS << "Verifying unit: " << Index << " / " << Units.getNumUnits();
-    if (const char* Name = Unit->getUnitDIE(true).getShortName())
+    if (const char *Name = Unit->getUnitDIE(true).getShortName())
       OS << ", \"" << Name << '\"';
     OS << '\n';
     OS.flush();
@@ -539,14 +541,12 @@ bool DWARFVerifier::handleDebugInfo() {
   unsigned NumErrors = 0;
 
   OS << "Verifying .debug_info Unit Header Chain...\n";
-  DObj.forEachInfoSections([&](const DWARFSection &S) {
-    NumErrors += verifyUnitSection(S);
-  });
+  DObj.forEachInfoSections(
+      [&](const DWARFSection &S) { NumErrors += verifyUnitSection(S); });
 
   OS << "Verifying .debug_types Unit Header Chain...\n";
-  DObj.forEachTypesSections([&](const DWARFSection &S) {
-    NumErrors += verifyUnitSection(S);
-  });
+  DObj.forEachTypesSections(
+      [&](const DWARFSection &S) { NumErrors += verifyUnitSection(S); });
 
   OS << "Verifying non-dwo Units...\n";
   NumErrors += verifyUnits(DCtx.getNormalUnitsVector());
@@ -922,8 +922,7 @@ unsigned DWARFVerifier::verifyDebugInfoReferences(
     return DWARFDie();
   };
   unsigned NumErrors = 0;
-  for (const std::pair<const uint64_t, std::set<uint64_t>> &Pair :
-       References) {
+  for (const std::pair<const uint64_t, std::set<uint64_t>> &Pair : References) {
     if (GetDIEForOffset(Pair.first))
       continue;
     ++NumErrors;
@@ -1434,8 +1433,8 @@ unsigned DWARFVerifier::verifyNameIndexAttribute(
   }
 
   if (AttrEnc.Index == dwarf::DW_IDX_parent) {
-    static constexpr dwarf::Form AllowedForms[] = {dwarf::Form::DW_FORM_flag_present,
-                                                   dwarf::Form::DW_FORM_ref4};
+    static constexpr dwarf::Form AllowedForms[] = {
+        dwarf::Form::DW_FORM_flag_present, dwarf::Form::DW_FORM_ref4};
     if (!is_contained(AllowedForms, AttrEnc.Form)) {
       ErrorCategory.Report("Unexpected NameIndex Abbreviation", [&]() {
         error() << formatv(
@@ -2125,7 +2124,8 @@ bool DWARFVerifier::verifyDebugStrOffsets(
       });
       Success = false;
     }
-    for (uint64_t Index = 0; C && C.tell() + OffsetByteSize <= NextUnit; ++Index) {
+    for (uint64_t Index = 0; C && C.tell() + OffsetByteSize <= NextUnit;
+         ++Index) {
       uint64_t OffOff = C.tell();
       uint64_t StrOff = DA.getAddress(C);
       // check StrOff refers to the start of a string

mlir/include/mlir/Dialect/Quant/IR/QuantTypes.h

@@ -401,8 +401,9 @@ class UniformQuantizedPerAxisType
 };
 
 /// QuantileQuantizedType derives from UniformQuantizedType and adds to it a
-/// look up table array of quantile values. The type of the data in the look up table is determined by
-/// the quantileType member: supported quantileType types are integer/unsigned/hf8/bf8/f16/bf16/f32/f64.
+/// look up table array of quantile values. The type of the data in the look up
+/// table is determined by the quantileType member: supported quantileType types
+/// are integer/unsigned/hf8/bf8/f16/bf16/f32/f64.
 ///
 /// Syntax synopsis:
 ///   Per-layer, all parameters expressed:
@@ -441,12 +442,11 @@ class QuantileQuantizedType
              int64_t storageTypeMin, int64_t storageTypeMax);
 
   /// Verifies construction invariants and issues errors/warnings.
-  static LogicalResult verifyInvariants(function_ref<InFlightDiagnostic()> emitError,
-                              unsigned flags, Type storageType,
-                              Type quantileType, Type expressedType,
-                              ArrayRef<double> quantiles, double scale,
-                              int64_t zeroPoint, int64_t storageTypeMin,
-                              int64_t storageTypeMax);
+  static LogicalResult
+  verifyInvariants(function_ref<InFlightDiagnostic()> emitError, unsigned flags,
+                   Type storageType, Type quantileType, Type expressedType,
+                   ArrayRef<double> quantiles, double scale, int64_t zeroPoint,
+                   int64_t storageTypeMin, int64_t storageTypeMax);
 
   static bool classof(mlir::Type type);
 
@@ -468,8 +468,9 @@ class QuantileQuantizedType
 };
 
 /// Represents per-axis QuantileQuantizedType (also known as per-channel
-/// quantization). The type of the data in the look up table is determined by the
-/// quantileType member: supported quantileType types are integer/unsigned/hf8/bf8/f16/bf16/f32/f64.
+/// quantization). The type of the data in the look up table is determined by
+/// the quantileType member: supported quantileType types are
+/// integer/unsigned/hf8/bf8/f16/bf16/f32/f64.
 ///
 /// Syntax synopsis:
 ///   Per-axis, all parameters expressed:
@@ -516,10 +517,10 @@ class QuantileQuantizedPerAxisType
   /// Verifies construction invariants and issues errors/warnings.
   static LogicalResult
   verifyInvariants(function_ref<InFlightDiagnostic()> emitError, unsigned flags,
-         Type storageType, Type quantileType, Type expressedType,
-         ArrayRef<double> quantiles, ArrayRef<double> scales,
-         ArrayRef<int64_t> zeroPoints, int32_t quantizedDimension,
-         int64_t storageTypeMin, int64_t storageTypeMax);
+                   Type storageType, Type quantileType, Type expressedType,
+                   ArrayRef<double> quantiles, ArrayRef<double> scales,
+                   ArrayRef<int64_t> zeroPoints, int32_t quantizedDimension,
+                   int64_t storageTypeMin, int64_t storageTypeMax);
 
   static bool classof(mlir::Type type);
 

mlir/include/mlir/IR/BuiltinTypes.h

@@ -250,7 +250,7 @@ enum class SliceVerificationResult {
 /// code.
 SliceVerificationResult isRankReducedType(ShapedType originalType,
                                           ShapedType candidateReducedType);
-                                          
+
 //===----------------------------------------------------------------------===//
 // Convenience wrappers for VectorType
 //

mlir/include/mlir/IR/OpImplementation.h

@@ -119,10 +119,10 @@ class AsmPrinter {
   /// Print a newline and indent the printer to the start of the current
   /// operation.
   virtual void printNewline();
-  
+
   /// Increase indentation.
   virtual void increaseIndent();
-  
+
   /// Decrease indentation.
   virtual void decreaseIndent();
 

mlir/include/mlir/IR/SymbolTable.h

@@ -265,6 +265,7 @@ class SymbolTable {
   static LogicalResult replaceAllSymbolUses(Operation *oldSymbol,
                                             SymbolRefAttr newSymbolName,
                                             Region *from);
+
 private:
   Operation *symbolTableOp;
 
@@ -474,10 +475,10 @@ class SymbolTable : public TraitBase<ConcreteType, SymbolTable> {
 
 template <typename ConcreteType>
 class SymbolContainer : public TraitBase<ConcreteType, SymbolContainer> {
-  public:
-    static LogicalResult verifyTrait(Operation *op) {
-      return mlir::success(); //TODO::implement
-    }
+public:
+  static LogicalResult verifyTrait(Operation *op) {
+    return mlir::success(); // TODO::implement
+  }
 
   /// Look up a symbol with the specified name, returning null if no such
   /// name exists. Symbol names never include the @ on them. Note: This

mlir/include/mlir/Interfaces/SideEffectInterfaces.h

@@ -146,17 +146,17 @@ class EffectInstance {
       : effect(effect), resource(resource), stage(stage),
         effectOnFullRegion(effectOnFullRegion) {}
   template <typename T,
-            std::enable_if_t<
-                llvm::is_one_of<T, OpOperand *, OpResult, BlockArgument, Value>::value,
-                bool> = true>
+            std::enable_if_t<llvm::is_one_of<T, OpOperand *, OpResult,
+                                             BlockArgument, Value>::value,
+                             bool> = true>
   EffectInstance(EffectT *effect, T value,
                  Resource *resource = DefaultResource::get())
       : effect(effect), resource(resource), value(value), stage(0),
         effectOnFullRegion(false) {}
   template <typename T,
-            std::enable_if_t<
-                llvm::is_one_of<T, OpOperand *, OpResult, BlockArgument, Value>::value,
-                bool> = true>
+            std::enable_if_t<llvm::is_one_of<T, OpOperand *, OpResult,
+                                             BlockArgument, Value>::value,
+                             bool> = true>
   EffectInstance(EffectT *effect, T value, int stage, bool effectOnFullRegion,
                  Resource *resource = DefaultResource::get())
       : effect(effect), resource(resource), value(value), stage(stage),
@@ -267,7 +267,8 @@ class EffectInstance {
 
   /// The Symbol, OpOperand, OpResult or BlockArgument that the effect applies
   /// to. This is optionally null.
-  PointerUnion<SymbolRefAttr, OpOperand *, OpResult, BlockArgument, Value> value;
+  PointerUnion<SymbolRefAttr, OpOperand *, OpResult, BlockArgument, Value>
+      value;
 
   /// Additional parameters of the effect instance. An attribute is used for
   /// type-safe structured storage and context-based uniquing. Concrete effects

mlir/include/mlir/Pass/Pass.h

@@ -184,8 +184,9 @@ class Pass {
   /// invoked within this hook.
   /// This method is invoked after all dependent dialects for the pipeline are
   /// loaded, and is not allowed to load any further dialects (override the
-  /// `getDependentDialects()` for this purpose instead). Returns a LogicalResult
-  /// to indicate failure, in which case the pass pipeline won't execute.
+  /// `getDependentDialects()` for this purpose instead). Returns a
+  /// LogicalResult to indicate failure, in which case the pass pipeline won't
+  /// execute.
   virtual LogicalResult initialize(MLIRContext *context) { return success(); }
 
   /// Indicate if the current pass can be scheduled on the given operation type.
@@ -296,7 +297,7 @@ class Pass {
   /// Copy the option values from 'other', which is another instance of this
   /// pass.
   void copyOptionValuesFrom(const Pass *other);
-  
+
   /// Copy the option values from 'other', which are PassPipeline options.
   /// Here we copy only those options that have the same argument name.
   void copyOptionValuesFrom(const detail::PassOptions &other);

mlir/include/mlir/Pass/PassOptions.h

@@ -342,7 +342,7 @@ class PassOptions : protected llvm::cl::SubCommand {
   /// Copy the option values from 'other' into 'this', where 'other' has the
   /// same options as 'this'.
   void copyOptionValuesFrom(const PassOptions &other);
-  
+
   /// Copy only those options that have the same argument name.
   void matchAndCopyOptionValuesFrom(const PassOptions &otherPassOptions);
 

mlir/include/mlir/Tools/mlir-opt/MlirOptMain.h

@@ -297,7 +297,7 @@ class MlirOptMainConfig {
 /// VPUX-specific method to get value for arch kind from command line
 /// and register HW-specific passes and pipelines
 using AdditionalRegistrationFn =
-      std::function<void(llvm::StringRef helpHeader)>;
+    std::function<void(llvm::StringRef helpHeader)>;
 
 /// This defines the function type used to setup the pass manager. This can be
 /// used to pass in a callback to setup a default pass pipeline to be applied on
@@ -328,10 +328,10 @@ LogicalResult MlirOptMain(llvm::raw_ostream &outputStream,
 /// - registry should contain all the dialects that can be parsed in the source.
 /// - additionalRegistration will be called before the main command line parsing
 ///   to perform additional registrations.
-LogicalResult MlirOptMain(int argc, char **argv, llvm::StringRef toolName,
-                          DialectRegistry &registry,
-                          const AdditionalRegistrationFn &additionalRegistration
-                                  = [](llvm::StringRef){});
+LogicalResult MlirOptMain(
+    int argc, char **argv, llvm::StringRef toolName, DialectRegistry &registry,
+    const AdditionalRegistrationFn &additionalRegistration =
+        [](llvm::StringRef) {});
 
 /// Helper wrapper to return the result of MlirOptMain directly from main.
 ///