[Mlir-commits] [mlir] 18e33ba - Remove `mutable` and stray comment
Sanjoy Das
llvmlistbot at llvm.org
Mon Jan 31 12:44:17 PST 2022
Author: Sanjoy Das
Date: 2022-01-31T12:33:20-08:00
New Revision: 18e33ba9c94e7888450349025f6b6bff9f82ed5f
URL: https://github.com/llvm/llvm-project/commit/18e33ba9c94e7888450349025f6b6bff9f82ed5f
DIFF: https://github.com/llvm/llvm-project/commit/18e33ba9c94e7888450349025f6b6bff9f82ed5f.diff
LOG: Remove `mutable` and stray comment
The `mutable` was added back when `scope` was a `DataLayoutOpInterface`.
Differential Revision: https://reviews.llvm.org/D118643
Added:
Modified:
mlir/include/mlir/Interfaces/DataLayoutInterfaces.h
Removed:
################################################################################
diff --git a/mlir/include/mlir/Interfaces/DataLayoutInterfaces.h b/mlir/include/mlir/Interfaces/DataLayoutInterfaces.h
index 6d214ed30236..5cc80a06f363 100644
--- a/mlir/include/mlir/Interfaces/DataLayoutInterfaces.h
+++ b/mlir/include/mlir/Interfaces/DataLayoutInterfaces.h
@@ -173,9 +173,7 @@ class DataLayout {
void checkValid() const;
/// Operation defining the scope of requests.
- // TODO: this is mutable because the generated interface method are not const.
- // Update the generator to support const methods and change this to const.
- mutable Operation *scope;
+ Operation *scope;
/// Caches for individual requests.
mutable DenseMap<Type, unsigned> sizes;
More information about the Mlir-commits
mailing list