[PATCH] D42357: Under limitation of allocated buffer, inplace_merge() does NOT take usage of partial allocated buffer but applies native rotate directly.
WeiChungChang via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Jan 24 15:46:21 PST 2018
WeiChungChang added a comment.
Here gives the comparison after fix.
F5783231: o1.png <https://reviews.llvm.org/F5783231>
It is better now but still slower; also, one could notice the standard variance (each round the execution time is unstable even the input is uniform)
The problem will be shown on the other ticket.
| | full | 25% | 12.5% | 6.25% | 3.125% |
| LLVM Opt 1 | 0.2401988 | 0.407111 | 0.8510433 | 1.2216213 | 1.3857887 |
| LLVM | 0.238614667 | 0.733295778 | 1.137285111 | 1.348671667 | 1.603639556 |
| GCC | 0.368778444 | 0.351659222 | 0.532976333 | 0.660785778 | 0.832495444 |
|
Repository:
rCXX libc++
https://reviews.llvm.org/D42357
More information about the cfe-commits
mailing list