[PATCH] D119051: Fix pod-packed functionality to use the C++11 definition of pod-ness

Bhramar Vatsa via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Feb 5 06:43:22 PST 2022


Bhramar.vatsa added inline comments.


================
Comment at: clang/lib/AST/RecordLayoutBuilder.cpp:1892
+  bool FieldPacked =
+      (Packed && (!FieldClass || D->getType().isCXX11PODType(Context) ||
+                  Context.getLangOpts().getClangABICompat() <=
----------------
Maybe irrespective of the field's type itself, only important thing is just if it is a POD type or not?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D119051



More information about the cfe-commits mailing list