mirror of
https://github.com/c64scene-ar/llvm-6502.git
synced 2025-01-12 02:33:33 +00:00
Added new FIXME note
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@129360 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
parent
d0aacbcc2e
commit
18bba84d77
@ -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 <cstdio>
|
||||
|
||||
#include <sstream>
|
||||
#include <stdexcept>
|
||||
|
||||
|
Loading…
x
Reference in New Issue
Block a user