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

Chris Lattner via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 12 14:27:39 PST 2016


On Feb 12, 2016, at 2:24 PM, Philip Reames <listmail at philipreames.com> wrote:
> 
>>> 
>>> That said, both of those patches are irrelevant to the discussion, because Argyrios removed the swift header and replaced it with the LLVM header when he checked it in.
>> 
>> Incidentally and unrelatedly, the Swift repo has a bunch of other generally useful “ADT like” support stuff:
>> https://github.com/apple/swift/tree/master/include/swift/Basic <https://github.com/apple/swift/tree/master/include/swift/Basic>
>> If folks are interested in seeing any parts of it go up to LLVM, I’d love to see that happen.
> I'd love to see relevant pieces moved over - provided we actually use them in LLVM - but we need to make sure we dot the "i"s and cross the "t"s on the licensing part.  Tedious, but necessary.

Yes yes, you don’t need to convince me about the merits of doing things right from a licensing perspective, seriously. :-)

My question is whether there were any specific things people would make sense and people would *want* sunk down to LLVM either ADT, or other things in Swift.  If there is an interest, we can make sure the right licensing things happens.

-Chris

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160212/0fb51a3c/attachment.html>


More information about the llvm-commits mailing list