[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