[PATCH] D18656: [SimplifyLibCalls] Strip dead code in printf() transformations

[Filipe Cabecinhas via llvm-commits]

Do you want to add an assert somewhere?
Just in case :-)

Thank you,

  Filipe

On Mon, Apr 4, 2016 at 1:09 PM, Davide Italiano <dccitaliano at gmail.com> wrote:
> davide added a comment.
>
> I assume that might be a problem.
> This change is NFC because the first expression always evaluate to true so we never care about the !Res case even prior to this change. In case we hit a case you describe I'll be more than happy to fix but I think the possibility of an environment defining printf() and not putchar() is very remote anyway.
>
>
> Repository:
>   rL LLVM
>
> http://reviews.llvm.org/D18656
>
>
>