[PATCH] D36922: [LibCallSimplifier] try harder to fold memcmp with constant arguments
Eli Friedman via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Aug 23 13:03:48 PDT 2017
efriedma added inline comments.
================
Comment at: llvm/trunk/lib/Transforms/Utils/SimplifyLibCalls.cpp:778
IntType->getPointerTo(LHS->getType()->getPointerAddressSpace());
+ LHSV = B.CreateLoad(B.CreateBitCast(LHS, LHSPtrTy), "lhsv");
+ }
----------------
Are you sure this is right? It looks like it you'll create a dead load if the LHS is aligned, but the RHS isn't (and therefore drive instcombine into an infinite loop).
Repository:
rL LLVM
https://reviews.llvm.org/D36922
More information about the llvm-commits
mailing list