[Mlir-commits] [mlir] [mlir][math] Add vector support for math-to-apfloat (PR #172715)
Maksim Levental
llvmlistbot at llvm.org
Thu Jan 15 15:57:48 PST 2026
makslevental wrote:
> My one concern is that you're storing ArrayRef, which will have lifetime issues - I'd copy those into a SmallVector instead
I don't think this is a big deal - the backing for the `ArrayRef` outlives all the Patterns because the patterns are added to a `PatternSet` "owned" by the pass (ie created in `runOnOperation`). If we expose the patterns via "populators" it'll still be the same thing (whichever pass adds the patterns will have its backing which outlives its own `PatternSet`).
https://github.com/llvm/llvm-project/pull/172715
More information about the Mlir-commits
mailing list