From a0f6ecb01f6320b73e2a285066320d2f6ea6d69d Mon Sep 17 00:00:00 2001 From: Garrison Venn Date: Tue, 12 Apr 2011 12:30:10 +0000 Subject: Added new FIXME note llvm-svn: 129360 --- llvm/examples/ExceptionDemo/ExceptionDemo.cpp | 9 +++++++++ 1 file changed, 9 insertions(+) (limited to 'llvm/examples') diff --git a/llvm/examples/ExceptionDemo/ExceptionDemo.cpp b/llvm/examples/ExceptionDemo/ExceptionDemo.cpp index fcc8b5c790d..5fa10ed6d35 100644 --- a/llvm/examples/ExceptionDemo/ExceptionDemo.cpp +++ b/llvm/examples/ExceptionDemo/ExceptionDemo.cpp @@ -62,7 +62,16 @@ #include "llvm/Support/IRBuilder.h" #include "llvm/Support/Dwarf.h" +// FIXME: Although all systems tested with (Linux, OS X), do not need this +// header file included. A user on ubuntu reported, undefined symbols +// for stderr, and fprintf, and the addition of this include fixed the +// issue for them. Given that LLVM's best practices include the goal +// of reducing the number of redundant header files included, the +// correct solution would be to find out why these symbols are not +// defined for the system in question, and fix the issue by finding out +// which LLVM header file, if any, would include these symbols. #include + #include #include -- cgit v1.2.3