[PATCH] D34921: [ConstantHoisting] Remove dupliate logic in constant hoisting
Eli Friedman via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jul 5 14:13:44 PDT 2017
efriedma added inline comments.
================
Comment at: lib/Transforms/Scalar/ConstantHoisting.cpp:412
+ // only and intrinsics which can not be freely called with constants. So
+ // it's safe for us to collect constant candidates from all IntrinsicInsts.
+ if (canReplaceOperandWithVariable(Inst, Idx)
----------------
Are you sure this is true...? For example, llvm.memcpy has an "len" parameter which can be hoisted, and an "align" parameter which can't.
Or maybe I'm misinterpreting what this comment is talking about?
https://reviews.llvm.org/D34921
More information about the llvm-commits
mailing list