[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