[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