[Mlir-commits] [mlir] [mlir][spirv] Fix UpdateVCEPass to deduce the correct set of capabilities (PR #151108)
Igor Wodiany
llvmlistbot at llvm.org
Wed Jul 30 08:41:23 PDT 2025
IgWod-IMG wrote:
I cannot quickly run asan on my machine, but as a proxy I ran Valgrind and it highlights some invalid reads in the new function, so this may be causing builbot issues.
```
...
==1419720== Thread 4 llvm-worker-2:
==1419720== Invalid read of size 4
==1419720== at 0x5272660: addAllImpliedCapabilities(llvm::SetVector<mlir::spirv::Capability, llvm::SmallVector<mlir::spirv::Capability, 0u>, llvm::DenseSet<mlir::spirv::Capability, llvm::DenseMapInfo<mlir::spirv::Capability, void> >, 0u>&) (llvm-project/mlir/lib/Dialect/SPIRV/Transforms/UpdateVCEPass.cpp:99)
...
```
@kuhar does it need to be reverted?
https://github.com/llvm/llvm-project/pull/151108
More information about the Mlir-commits
mailing list