[clang] [CIR] Correct union lowering behavior re-padding (PR #216349)

Andy Kaylor via cfe-commits cfe-commits at lists.llvm.org
Fri Aug 14 16:48:41 PDT 2026


================
@@ -416,28 +416,68 @@ static bool shouldPackFAMStruct(const mlir::DataLayout &dataLayout,
 // Additionally, the struct itself could contain a struct with a FAM or a union
 // that needed adjustment, so it recurses to check those.  If no such type has
 // been found/no adjustment needed, this returns the type unchanged.
+//
+// Additionally, a union having an active member of the not-the-largest
----------------
andykaylor wrote:

```suggestion
// Additionally, a union having an active member of not-the-largest
```
I'm not sure about this. It read awkwardly to me.

https://github.com/llvm/llvm-project/pull/216349


More information about the cfe-commits mailing list