[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