[PATCH] D50233: [InstCombine] Transform str(n)cmp to memcmp

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 5 11:54:23 PDT 2018


efriedma added a comment.

If it were just msan, I would say we could just check for the msan attribute and call it done.  But I'm a little worried about the potential race and potential weird interactions with LTO.  So probably best to both bailout if msan is enabled, and move the transform to expandmemcmp.


Repository:
  rL LLVM

https://reviews.llvm.org/D50233





More information about the llvm-commits mailing list