[Mlir-commits] [mlir] Revert "[mlir] Fix a warning" (PR #73640)
Kazu Hirata
llvmlistbot at llvm.org
Tue Dec 12 15:14:42 PST 2023
kazutakahirata wrote:
Sorry for the slow response here. If we simply revert the patch, we would get the same warning as before:
```
mlir/lib/Pass/PassRegistry.cpp:376:37: error: ISO C++ requires the name after '::~' to be found in the same scope as the name before '::~' [-Werror,-Wdtor-name]
```
Does this mean we need to check the C++ version like so?
```
#if __cplusplus >= 202002L
llvm::cl::OptionValue<OpPassManager>::~OptionValue<OpPassManager>() = default;
#else
llvm::cl::OptionValue<OpPassManager>::~OptionValue() = default;
#endif
```
https://github.com/llvm/llvm-project/pull/73640
More information about the Mlir-commits
mailing list