[PATCH] D105440: [RFC] Implement support for __builtin_memcmp_inline

[Guillaume Chatelet via Phabricator via llvm-commits]

gchatelet added a comment.

> I initially benchmarked memcmp using the scalar building blocks and I was seeing better codegen with the builtins. I couldn't quite remember what the differences were so I decided to benchmark the scalar implementations again and they seem to perform fairly similar now, there is a small codechange around size 3, but the codegen actually seems better to me with the scalar implementation. I may have been using an older llvm or an earlier version of the building blocks.
> I am happy to drop this RFC if there are no objections.

Ok then let's drop it.
I think we still need to improve the state of `memcmp` in LLVM but let's decouple the two issues.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D105440/new/

https://reviews.llvm.org/D105440