[PATCH] Introduce llvm/ADT/OptionSet.h utility class

David Blaikie via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 11 23:26:48 PST 2016


It's generally considered poor form to commit something without review,
after asking for it...

On Thu, Feb 11, 2016 at 6:54 PM, Argyrios Kyrtzidis via llvm-commits <
llvm-commits at lists.llvm.org> wrote:

> I’ve took the liberty of committing in r260652, since this is general
> goodness and doesn’t affect existing code.
>
> > On Feb 10, 2016, at 9:23 PM, Argyrios Kyrtzidis <kyrtzidis at apple.com>
> wrote:
> >
> > Hi all,
> >
> > The attached patch introduces ‘OptionSet’ in llvm/ADT headers, which is
> a utility class that makes it convenient to work with enumerators
> representing bit options.
> >
> > We have found it useful in the swift repo (
> https://github.com/apple/swift/blob/master/include/swift/Basic/OptionSet.h)
> and I think it will be useful to other llvm projects as well.
> >
> > Let me know if you think it is ok to commit in llvm repo.
> >
> > <optionset.patch>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160211/d217e728/attachment.html>


More information about the llvm-commits mailing list