[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