[PATCH] D30629: [APInt] Give the value union a name so we can remove assumptions on VAL being the larger member

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 6 15:33:03 PST 2017


dblaikie accepted this revision.
dblaikie added a comment.
This revision is now accepted and ready to land.

Skimmed through most of it - looks good to me.

If you like, you could wait to see, or check with others, in case my idea that this is the Right Way to do this (having a named enum, etc) is correct/a shared opinion.


https://reviews.llvm.org/D30629





More information about the llvm-commits mailing list