C++11 definition of POD and tail padding

Hi,

Attached is a testcase that will or will not use tail padding depending on the
language mode (C++03 versus C++11). The test uses a class that is POD in C++11
but non-POD in C++03. Using a recent TOT of compiler (clang r185419), the
compiler will use the tail padding in C++03 mode but will not use it in C++11
mode.

The Itanium C++ ABI (http://mentorembedded.github.io/cxx-abi/abi.html) has a
clause which states that for the purpose of layout the C++03 definition of POD
should be used:

" POD for the purpose of layout

  ...

  There have been multiple published revisions to the ISO C++ standard,
  and each one has included a different definition of POD. To ensure
  interoperation of code compiled according to different revisions of
  the standard, it is necessary to settle on a single definition for a
  platform. A platform vendor may choose to follow a different revision
  of the standard, but by default, the definition of POD under this ABI
  is the definition from the 2003 revision (TC1).
"

Furthermore, r173744 refactored the RecordLayoutBuilder code and added comments
which suggest that Clang should be following the Itanium ABI with respect
towards the above POD layout rules:
"
// To preserve binary compatibility, the generic Itanium ABI has
// permanently locked the definition of POD to the rules of C++ TR1,
// and that trickles down to all the derived ABIs.
"

Given this, I would expect the attached testcase to use the tail padding even
in C++11 mode. It seems that code responsible for determining if tail padding
should be used (mustSkipTailPadding) expects that isPOD() checks the C++ TR1
(2003) definition of POD, however the behaviour with this testcase shows that
isPOD is not using the TR1 definition in all cases.

I also attached a small sample patch that at least superficially fixes the
problem by explicitly calling isCXX98PODType() instead of isPODType() when
setting data().PlainOldData (which is what isPOD() returns.) The patch is not
presented as a real fix, but just to help demonstrate why isPOD() is not
consistently using the TR1 definition of POD.

Should I file a bug about this?

Thanks,
Josh

tail-padding.cpp (880 Bytes)

podtype.diff (503 Bytes)

It looks like a bug, yes, please file a bug.

cheers,
--renato

Filed here: http://llvm.org/bugs/show_bug.cgi?id=16537.

Thanks,
- Josh