[llvm-commits] [llvm] r80710 - in /llvm/trunk/lib: CodeGen/AsmPrinter/DwarfException.cpp Target/ARM/AsmPrinter/ARMAsmPrinter.cpp

Daniel Dunbar daniel at zuster.org
Thu Sep 3 00:46:16 PDT 2009


On Wed, Sep 2, 2009 at 10:24 PM, Chris Lattner<clattner at apple.com> wrote:
> On Sep 2, 2009, at 10:01 PM, Daniel Dunbar wrote:
>> On Wed, Sep 2, 2009 at 9:59 PM, Daniel Dunbar<daniel at zuster.org> wrote:
>>>
>>> I'm happy to work on fixing this if someone has a good plan.
>>> raw_ostream doesn't really seem to need its 64 byte buffer, for
>>> example. When I first implemented
>>
>> ... the magic I-ownz-your-buffer ..., that is.
>
> Ah, ok, bumping it up to 64 with a comment about this seems reasonable.

That seems burdensome for all callers, we should probably fix the API.
What if we dropped the raw_ostream required buffer size to 4, since I
think thats actually all that is required?

 - Daniel



More information about the llvm-commits mailing list