[PATCH] [InstCombine] Propagate non-null facts to call parameters
Pete Cooper
peter_cooper at apple.com
Mon Jun 15 18:05:37 PDT 2015
Sorry, Phab is acting up for me, hence email.
I think this line should actually set 'Changed = true;’ and not return. That way you can handle more than a single nonnull at a time.
+ return CS.getInstruction();
Otherwise LGTM.
Cheers,
Pete
REPOSITORY
rL LLVM
http://reviews.llvm.org/D9132
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the llvm-commits
mailing list