[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