[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