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 b8bf14e commit fa733c2
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions mlir/include/mlir/IR/StorageUniquerSupport.h
Original file line number Diff line number Diff line change
Expand Up @@ -227,8 +227,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) {
verifyInvariants(Args... args) {
return success();
}

Expand Down

0 comments on commit fa733c2

Please sign in to comment.