[PATCH] D40291: [Support] Add WritableMemoryBuffer class

David Blaikie via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 15 13:42:31 PST 2017


Fair - there might be some complexities I'm not considering, but figured it
was worth asking/taking at least a brief look at to understand the design
space - thanks!

On Fri, Dec 15, 2017 at 1:34 PM Pavel Labath <labath at google.com> wrote:

> On 15 December 2017 at 19:22, David Blaikie <dblaikie at gmail.com> wrote:
> > What if the two existing derived classes were templated on their base
> class?
> > (so you could have MemoryBufferMem<WritableMemoryBuffer> and
> > MemoryBufferMem<MemoryBuffer>) would that be reasonable?
> >
>
> Yeah, that's the template trickery I was referring to. :)
>
> It can definitely be made to work, but I am not sure how nice will the
> end result be. I can give it a shot next week.
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171215/be9e5515/attachment.html>


More information about the llvm-commits mailing list