[PATCH] Do not optimise fprintf() calls if its return value is used.

Meador Inge meadori at codesourcery.com
Mon Apr 8 09:25:29 PDT 2013


On 04/03/2013 05:10 PM, Peter Collingbourne wrote:

> http://llvm-reviews.chandlerc.com/D620
> 
> Files:
>   lib/Transforms/Utils/SimplifyLibCalls.cpp
>   test/Transforms/InstCombine/fprintf-1.ll

This patch is OK.  Thanks for fixing it.

I know we have similar logic in place for the printf simplifiers.
Did you audit the code to see if we have other return value use
bugs?


-- 
Meador Inge
CodeSourcery / Mentor Embedded



More information about the llvm-commits mailing list