[PATCH] D26942: [SelectionDAG] getRawSubclassData should not return HasDebugValue.
Chandler Carruth via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Nov 28 11:42:50 PST 2016
chandlerc added inline comments.
================
Comment at: include/llvm/CodeGen/SelectionDAGNodes.h:475-488
+ union AllSDNodeBitfields {
+ uint16_t Value;
SDNodeBitfields SDNodeBits;
ConstantSDNodeBitfields ConstantSDNodeBits;
MemSDNodeBitfields MemSDNodeBits;
LSBaseSDNodeBitfields LSBaseSDNodeBits;
LoadSDNodeBitfields LoadSDNodeBits;
----------------
This change seems a really big change and completely outside of the scope I suggested. I don't particularly like it (it seems to make things less clear) and I'm not sure what motivates it. Could you instead try the approach I suggested?
https://reviews.llvm.org/D26942
More information about the llvm-commits
mailing list