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

Sanjay Patel via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 4 10:22:50 PDT 2016


spatel added a comment.

In http://reviews.llvm.org/D18656#390825, @filcab wrote:

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


I think we're already safe here. On failure (eg, putchar is not available), emitPutChar() returns nullptr, so optimizePrintFString() will also return nullptr to signal to its caller that the transform didn't work.


Repository:
  rL LLVM

http://reviews.llvm.org/D18656





More information about the llvm-commits mailing list