[PATCH] D119051: Extend the C++03 definition of POD to include defaulted functions
Reid Kleckner via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Feb 28 16:31:25 PST 2022
rnk added a comment.
I would structure this as a LangOpt, and feed the target checks and ABI compat checks into the default setting for it. It could be something like `DefaultedSMFArePOD` / `-f[no-]defaulted-smf-are-pod` (smf being special member functions). The LangOpt default is true, and the driver fills in the value from the target and ABI version in the case of no explicit flag.
I think the DeclCXX changes are probably in the right place. I don't think it makes sense to carry around separate `isPOD` bits according to the C++11 and C++03 rules, just so we can use them later to make these two specific, known ABI-impacting decisions.
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