[PATCH] D87974: [Builtin] Add __builtin_zero_non_value_bits.

Olivier Giroux via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Dec 7 19:35:18 PST 2020


__simt__ added a comment.

I think Jonathan is asking whether there is a match in the gray areas.

The two cases people bring up most:

1. Unions, where the padding overlaps for all the possible active members.
2. Tail padding, up to the allocator granularity / alignment size.

If the implementation-specific builtins don't match on these, then maybe they should have different names, is his argument I think.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D87974/new/

https://reviews.llvm.org/D87974



More information about the cfe-commits mailing list