[PATCH] D16337: [LibCallSimplifier] fold memset(malloc(x), 0, x) --> calloc(1, x)
Sanjay Patel via llvm-commits
llvm-commits at lists.llvm.org
Mon Jan 25 16:13:44 PST 2016
spatel added a comment.
In http://reviews.llvm.org/D16337#334992, @mcrosier wrote:
> LGTM assuming you've done the necessary correctness/performance due diligence.
Thanks, Chad. I ran test-suite and see no fails (and don't expect this to fire anyway due to the 1-use restriction). I'm not able to measure any perf differences even with a micro-benchmark on OSX, but at least the code is smaller.
http://reviews.llvm.org/D16337
More information about the llvm-commits
mailing list