Don't define our own global 'endl' variable. While technically it had
authorChandler Carruth <chandlerc@gmail.com>
Mon, 8 Apr 2013 08:30:47 +0000 (08:30 +0000)
committerChandler Carruth <chandlerc@gmail.com>
Mon, 8 Apr 2013 08:30:47 +0000 (08:30 +0000)
commitbcff69a1e0788377b8bec87f850a6f5977fe9d22
treed1aa53a448fb208999f227471a191b9d2384f201
parent7fba6cd3d0a9565e86f50bd3d6ac403eadbcb00c
Don't define our own global 'endl' variable. While technically it had
internal linkage and so wasn't a patent bug, it doesn't make any sense
here. We can avoid even calling operator<< by just embedding the newline
in the string literals that were already being streamed out. It also
gives the impression of some line-ending agnosticisms which is not
present, and that flushing happens when it doesn't.

If we want to use std::endl, we could do that, but honestly it doesn't
seem remotely worth it. Using '\n' directly is much more clear when
working with raw_ostream.

It also happens to fix builds with old crufty GCC STL implementations
that include std::endl into the global namespace (or headers written to
be compatible with such atrocities).

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@179003 91177308-0d34-0410-b5e6-96231b3b80d8
tools/obj2yaml/obj2yaml.cpp