From b59b2a10b16e17dcf439bf973b7bbd5da65d3c25 Mon Sep 17 00:00:00 2001
From: Lucy Fox <lucyfox@google.com>
Date: Mon, 15 Jun 2020 15:08:04 -0700
Subject: [PATCH] [NFC] Add comment to specify status of FusedKernelMatcher
pass.
We're porting over this pass based on usage/need, so I'm updating the comments to make this clear.
PiperOrigin-RevId: 316553704
Change-Id: I5db9a4637edd3bcb27b4db6259b7113ae0f8dd0c
---
.../tensorflow/transforms/fused_kernel_matcher.cc | 14 +++++++++-----
1 file changed, 9 insertions(+), 5 deletions(-)
diff --git a/tensorflow/compiler/mlir/tensorflow/transforms/fused_kernel_matcher.cc b/tensorflow/compiler/mlir/tensorflow/transforms/fused_kernel_matcher.cc
index 3ccdd957798..4b10550df7b 100644
--- a/tensorflow/compiler/mlir/tensorflow/transforms/fused_kernel_matcher.cc
+++ b/tensorflow/compiler/mlir/tensorflow/transforms/fused_kernel_matcher.cc
@@ -32,11 +32,15 @@ namespace TF {
namespace {
-// Note: This implements fusions performed in the old Remapper Grappler pass.
-// That pass has specific cases for GPU and based on different target
-// configurations on both CPU and GPU (Intel MKL, ROCm, etc.). This MLIR pass
-// covers the general CPU case and at the moment does not account for any
-// target-specific configurations.
+// Note: This implements the fusions performed in the old Remapper Grappler
+// pass. That pass has specific cases for GPU and based on different
+// target configurations on both CPU and GPU (Intel MKL, ROCm, etc.). This MLIR
+// pass covers (some of) the general CPU case and at the moment does not account
+// for any target-specific configurations.
+
+// This pass is being ported over from the Grappler Remapper pass based on
+// need/usage. File a bug to request porting over additional fusions.
+
// TODO(b/158265178): Support GPU-specific fusions.
// TODO(b/158266710): Support CPU MKL configurations.