[PATCH] D117356: InstructionCombining: avoid eliding mismatched alloc/free pairs

Bryce Wilson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jan 16 12:26:23 PST 2022


Bryce-MW added a comment.

I just wanted to say, personally I like this idea a lot. Even if there is a better way to solve the specific problem in https://github.com/llvm/llvm-project/issues/40384, this overall gives more information which may be useful in the future. This is also part of what someone from Rust was asking for on the llvm-dev mailing list (which was the email that got me into contributing at all). I'll leave it up to the more experienced devs whether to approve this patch since they have a better idea of what they want the memory builtins system to look like but I do think it should be approved.


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

https://reviews.llvm.org/D117356



More information about the llvm-commits mailing list