[llvm] [IR] Make instr-create fns with Instruction *InsertBefore non-default (PR #86132)
Orlando Cazalet-Hyams via llvm-commits
llvm-commits at lists.llvm.org
Tue May 28 08:35:11 PDT 2024
https://github.com/OCHyams approved this pull request.
Mechanically LGTM
The changes look like they move in the direction agreed upon in the Discourse discussion. Making the `Instruction*` versions not the default makes it easier to deprecate and remove them.
It's a shame that this introduced some almost-duplicated functions in Instruction.cpp - OTOH, we plan to remove one version.
I think it's worth some others having this on their radar too (cc @dwblaikie and @nikic) - I couldn't see this linked in the Discourse thread, it's probably worth dropping a link there if I haven't just missed it.
https://github.com/llvm/llvm-project/pull/86132
More information about the llvm-commits
mailing list