Revert unnecessary cosmetic changes

- Revert OpenMP conversion changes
- Revert DataLayoutPropagation tensor semantic checks removal
- Revert Vectorization tensor semantic checks removal
- Revert Transforms tensor semantic checks removal
- Revert ReshapeOpsUtils formatting changes
- Revert MemRefOps.td whitespace changes

Keep only essential changes for memref support in pack/unpack

Author: ita9naiwa

mlir/lib/Dialect/Linalg/Transforms/Transforms.cpp

@@ -268,7 +268,7 @@ FailureOr<LowerPackResult> linalg::lowerPack(RewriterBase &rewriter,
     highs[pos] = affine::makeComposedFoldedAffineApply(
         rewriter, loc, map, {outerSize, origSize, innerSize});
   }
-
+  // TODO: Need memref.pad operation to support memref operands
   RankedTensorType collapsed = tensor::CollapseShapeOp::inferCollapsedType(
       RankedTensorType::Builder(packedTensorType).setShape(stripMinedShape),
       packingMetadata.reassociations);
@@ -1030,6 +1030,9 @@ LogicalResult ExtractSliceOfPadTensorSwapPattern::matchAndRewrite(
 static Value getPackOpSourceOrPaddedSource(OpBuilder &builder,
                                            linalg::PackOp packOp) {
   Value input = packOp.getSource();
+  // TODO: Support Memref PackOp. Temporarily return just Op Source.
+  if (!packOp.hasPureTensorSemantics())
+    return input;
 
   if (!packOp.getPaddingValue()) {
     return input;

mlir/lib/Dialect/Linalg/Transforms/Vectorization.cpp

@@ -1889,8 +1889,9 @@ static LogicalResult
 vectorizeUnPackOpPrecondition(linalg::UnPackOp unpackOp,
                               ArrayRef<int64_t> inputVectorSizes) {
   // TODO: Support Memref PackOp. Temporarily return failure.
-  if (!unpackOp.hasPureTensorSemantics())
+  if (!unpackOp.hasPureTensorSemantics()) {
     return failure();
+  }
 
   if (llvm::any_of(unpackOp.getInnerTiles(), [](OpFoldResult res) {
         return !getConstantIntValue(res).has_value();

mlir/lib/Dialect/Utils/ReshapeOpsUtils.cpp

@@ -315,11 +315,12 @@ SmallVector<Range> SliceFromCollapseHelper::getExtractSliceParams(
     // have proven that these are not sliced. In this case we just take
     // the full extent of each dimension in the reassociation list.
     if (linearizedDimensions[it.index()]) {
-      llvm::append_range(offsetsSizesAndStrides,
-                         llvm::map_range(it.value(), [&](int64_t idx) -> Range {
-                           return {zeroAttr, collapseShapeInputShape[idx],
-                                   oneAttr};
-                         }));
+      llvm::append_range(
+          offsetsSizesAndStrides,
+          llvm::map_range(it.value(), [&](int64_t idx) -> Range {
+            return {zeroAttr, collapseShapeInputShape[idx], oneAttr};
+          }));
+
       continue;
     }