While moving Args around, I noticed that we have a bunch of
functions/classes that pass/store version numbers as a triplet of integers
(e.g. Platform::GetOSVersion). I got halfway into creating a wrapper class
for that when I noticed clang::VersionTuple, which is pretty much what I
wanted out of the box.

Now there are small differences between this class, and what we have now:
it has an extra fourth "build" field, and it uses only 31 bits to represent
  the values. None of these seem to matter (particularly as we are
converting our representation into this struct in some places) that much,
but before I go through the trouble of pulling this class into llvm
(although technically possible, it seems wrong to pull a clang dependency
at such a low level), I wanted to make sure we are able to use it.

Do you see any reason why we could not replace our version triplets with
clang::VersionTuple ?

cheers,
pl
_______________________________________________
lldb-dev mailing list
lldb-dev@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-dev

Reply via email to