Skip to content

Commit

Permalink
[mlir][StorageUniquer] Restore old signature for default implementati…
Browse files Browse the repository at this point in the history
…on of verifyInvariants.

PR llvm#102326 changed the prototype of the default implementation of
verify to include emitErrorFn.

This breaks automatic derivation in consumer attributes, such as
https://github.com/tensorflow/runtime/blob/60277ba976739502e45ad26585e071568fa44af1/include/tfrt/core_runtime/opdefs/attributes.h#L53.

This PR simply restores the signature to what it was prior to PR llvm#102326.
  • Loading branch information
bchetioui committed Aug 13, 2024
1 parent e0ad56b commit 7f76489
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions mlir/include/mlir/IR/StorageUniquerSupport.h
Original file line number Diff line number Diff line change
Expand Up @@ -226,9 +226,7 @@ class StorageUserBase : public BaseT, public Traits<ConcreteT>... {

/// Default implementation that just returns success.
template <typename... Args>
static LogicalResult
verifyInvariants(function_ref<InFlightDiagnostic()> emitErrorFn,
Args... args) {
static LogicalResult verifyInvariants(Args... args) {
return success();
}

Expand Down

0 comments on commit 7f76489

Please sign in to comment.