commit 8f6b66f202d8bd3a7d002a9affbad55e635519ea
author Jacques Pienaar <jpienaar@google.com> Thu Sep 26 14:00:22 2019 -0700
committer TensorFlower Gardener <gardener@tensorflow.org> Thu Sep 26 15:43:24 2019 -0700
tree a523979296d7f7ad5bfd8cd1de5113d44ba34ec7
parent 5aa6c5a09b89e4764016a2a928e877be5e0e8276

Make result ops generated output deterministic

Sort the result ops reported in lexographical order.

PiperOrigin-RevId: 271426258

third_party/mlir/tools/mlir-tblgen/RewriterGen.cpp

diff --git a/third_party/mlir/tools/mlir-tblgen/RewriterGen.cpp b/third_party/mlir/tools/mlir-tblgen/RewriterGen.cpp
index 65453f0..7a125f5 100644
--- a/third_party/mlir/tools/mlir-tblgen/RewriterGen.cpp
+++ b/third_party/mlir/tools/mlir-tblgen/RewriterGen.cpp
@@ -417,7 +417,13 @@
   {0}(MLIRContext *context)
       : RewritePattern("{1}", {{)",
                 rewriteName, rootName);
-  interleaveComma(resultOps, os, [&](const Operator *op) {
+  // Sort result operators by name.
+  llvm::SmallVector<const Operator *, 4> sortedResultOps(resultOps.begin(),
+                                                         resultOps.end());
+  llvm::sort(sortedResultOps, [&](const Operator *lhs, const Operator *rhs) {
+    return lhs->getOperationName() < rhs->getOperationName();
+  });
+  interleaveComma(sortedResultOps, os, [&](const Operator *op) {
     os << '"' << op->getOperationName() << '"';
   });
   os << formatv(R"(}, {0}, context) {{})", pattern.getBenefit()) << "\n";