[PATCH] D128917: [SCEVExpander] Use IRBuilder to create BinOp

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 1 05:40:21 PDT 2022


nikic abandoned this revision.
nikic added a comment.

Yeah, I think I'm going to drop this for now. What would probably make most sense to me is that the caller of SCEVExpander can provide the builder, so that different users can use different configurations. But that would need some IRBuilder API changes, because we'd still want to use a custom inserter in either case.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D128917/new/

https://reviews.llvm.org/D128917



More information about the llvm-commits mailing list