[PATCH] D102679: [llvm] Make Sequence reverse-iterable

Tue Jun 8 10:23:40 PDT 2021

mehdi_amini added a comment.

Reverted ; seems like gcc5 didn't like some constructs in this patch, see details here: https://buildkite.com/mlir/mlir-core/builds/14326#2cda617f-aeb8-49d2-833c-b5a734cba86a ; let me know if you can't reproduce.

================
Comment at: mlir/lib/Dialect/Linalg/Transforms/Vectorization.cpp:321
-  SmallVector<int64_t> transposition(
-      llvm::seq<int64_t>(0, linalgOp.getNumLoops()));
   std::swap(transposition.back(), transposition[indexOp.dim()]);
----------------
Why did we lose this direct initialization pattern? Can we adjust something to keep this?

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D102679/new/

https://reviews.llvm.org/D102679