================
@@ -234,12 +234,19 @@ static bool isSupportedType(mlir::Type ty, const
DataLayout &dl) {
};
if (!llvm::any_of(members, spansRecord))
return false;
- // Classic sizes a union's coercion from the bytes that hold data, so
an
- // empty member contributes none. The library instead reduces the
union
- // to one member, picked by alignment and then by size, and coerces
from
- // that member: an empty one can win either comparison and widen the
- // coercion past what classic emits.
- if (llvm::any_of(members, memberIsEmptyRecord))
+ // A bit-field access unit's width can understate the bit-fields it
+ // holds. When the union has such a unit, a spanning member is not
----------------
adams381 wrote:
I've reworded to remove the term you didn't like. It should be more clear now.
https://github.com/llvm/llvm-project/pull/218718
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
