[PATCH] Implement CWG496: Is a volatile-qualified type really a POD?

Agustín Bergé kaballo86 at hotmail.com
Fri Jan 23 11:16:08 PST 2015


Addressed review comments. Added test for the newly introduced diagnostic.

CodeGenCXX/no-opt-volatile-memcpy.cpp still fails. I tried checking for the absence of `@llvm.memcpy`, but the non-volatile array member of the `struct` is still `memcpy`-ed (good!). I have little to no experience with codegen tests, let alone llvm ir, I could use some help to adjust the offending test case.


http://reviews.llvm.org/D7060

Files:
  include/clang/Basic/DiagnosticSemaKinds.td
  lib/AST/DeclCXX.cpp
  lib/Sema/SemaDeclCXX.cpp
  test/CXX/class/class.union/p1.cpp
  test/CXX/drs/dr4xx.cpp
  test/SemaCXX/type-traits.cpp

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D7060.18683.patch
Type: text/x-patch
Size: 6594 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20150123/9e02e8f9/attachment.bin>


More information about the cfe-commits mailing list