[PATCH] D80055: Diagnose union tail padding

JF Bastien via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri May 15 20:37:10 PDT 2020


jfb marked an inline comment as done.
jfb added inline comments.


================
Comment at: clang/include/clang/Basic/DiagnosticGroups.td:825
+def Padding : DiagGroup<"padding", [UnionTailPadding]>,
+              DiagCategory<"Padding Issue">;
+
----------------
I'd like to hear what other diagnostics folks think should fit under the `Padding` group. I've focused not on declarations which have padding, but rather on initializations which leave uninitialized padding behind. It seems like internal padding for structs and between array elements would be desirable.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80055





More information about the cfe-commits mailing list