[PATCH] D47214: [InstCombine] Replace bcmp/bcopy with memcmp/memmove
Sanjoy Das via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue May 29 10:19:39 PDT 2018
sanjoy added a comment.
In https://reviews.llvm.org/D47214#1114639, @Prazek wrote:
> Maybe try to come up with at least a microbenchmark that would show the gain. You can also benchmark LNT test suite and see if there is any improvement, but the microbenchmark should be easier.
Microbenchmarking is easier (almost every optimization can be shown to be "profitable" with the right microbenchmark :) ), but you'll have an easier time convincing us if there is an improvement in LNT (or any other "standard" benchmark really, like SPEC).
Repository:
rL LLVM
https://reviews.llvm.org/D47214
More information about the llvm-commits
mailing list