[LLVMdev] Handling of unsafe functions
Chris Lattner
clattner at apple.com
Fri Sep 21 09:39:51 PDT 2012
On Sep 20, 2012, at 3:08 PM, Richard Smith <richard at metafoo.co.uk> wrote:
> > I agree with Sean that we should move the code using C strings to LLVM
> > safe data types.
>
> I agree.
>
> >
> > And one more thing: it is interesting that the "unsafe"
> > APFloat::convertToHexString (from your patch) is not used anywhere.
>
> Zap it! Oh wait, is it used by Clang or something else?
>
> Clang doesn't use it, but LLDB does, with an arbitrary-seeming 256-character buffer. Perhaps we should change it to take an llvm::SmallVectorImpl<char>, rather than relying on a character buffer "which must be of sufficient size".
Makes sense to me,
-Chris
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20120921/2ebd1cbe/attachment.html>
More information about the llvm-dev
mailing list