[llvm-commits] Fix output stream opened in text instead of binary mode on MinGW
Danil Malyshev
dmalyshev at accesssoftek.com
Mon Nov 29 11:38:52 PST 2010
Hello Jakob,
> Can you find a way of doing this without increasing the size of the raw_ostream object, and without imposing any overhead on users that don't need it?
Ok, I did it.
Attached file fix-cout-02.diff is the new patch. It change only llvm-ar, llvm-ranlib, and attributes of *.toc files in Archive tests.
Thank you,
Danil
________________________________
From: Jakob Stoklund Olesen [mailto:stoklund at 2pi.dk]
Sent: Sunday, November 28, 2010 1:48 AM
To: Danil Malyshev
Cc: llvm-commits at cs.uiuc.edu
Subject: Re: [llvm-commits] Fix output stream opened in text instead of binary mode on MinGW
On Nov 24, 2010, at 9:07 AM, Danil Malyshev wrote:
Hi Danil,
Thanks for working on this.
1. Adds support for setw() method to the llvm::raw_ostream.
Please don't do that, though. Part of the idea behind raw_ostream is that it doesn't have any of the overhead from formatting and so on.
Can you find a way of doing this without increasing the size of the raw_ostream object, and without imposing any overhead on users that don't need it?
Thanks,
/jakob
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20101129/15b12036/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: fix-cout-02.diff
Type: application/octet-stream
Size: 6257 bytes
Desc: fix-cout-02.diff
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20101129/15b12036/attachment.obj>
More information about the llvm-commits
mailing list