[PATCH] D32712: [ConstantRange] Reduce the number of allocations in ConstantRange::makeGuaranteedNoWrapRegion

Davide Italiano via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 15 00:32:19 PDT 2017


davide requested changes to this revision.
davide added a comment.
This revision now requires changes to proceed.

This patch switches a one-liner with a much larger function.
Overall, I don't think this is necessarily unreasonable, but the increased complexity needs to be justified.
Craig, were you able to measure any measurable performance impact (in compile time) from the reduced malloc traffic?
If so, can you show the numbers?


https://reviews.llvm.org/D32712





More information about the llvm-commits mailing list