This updates the codebase to pass the context when creating an instance of
OwningRewritePatternList, and starts removing extraneous MLIRContext
parameters. There are many many more to be removed.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
Thanks Mehdi, I think this is a pretty obvious cleanup (and enables many others), so I'm going to land it.
Comment Actions
Looks like a nice cleanup, I see:
LowerToAffineLoops.cpp:299:28: error: no matching constructor for initialization of 'mlir::OwningRewritePatternList'
OwningRewritePatternList patterns; ^
in buildbot, was this fixed in follow up here? (my rebuild is being slow ...)
Comment Actions
I think Jacques already fixed, makes sure to be at HEAD before re-fixing the same one :)