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

[Davide Italiano via llvm-commits]

davide added inline comments.

================
Comment at: lib/Transforms/Utils/SimplifyLibCalls.cpp:1828-1832
@@ -1827,7 +1827,7 @@
 
   // Do not do any of the following transformations if the printf return value
   // is used, in general the printf return value is not compatible with either
   // putchar() or puts().
   if (!CI->use_empty())
     return nullptr;
 
----------------
spatel wrote:
> IMO, the asserts are unnecessary since the check for empty is right here and has a nice explanatory comment too.
> 
> If others like the added safety of the asserts, I don't object. There is a proposal to refactor this whole file, so maybe asserts will be handy.
> 
> But instead of repeating the assert 5x, add a one-line helper function or lambda?
My vote is to remove the asserts alltogheter. David?


http://reviews.llvm.org/D18656