rjmccall accepted this revision.
rjmccall added a comment.
This revision is now accepted and ready to land.

I wonder if that was originally just an oversight that got turned into official 
policy.  Anyway, if it's the policy, it's what we have to do; LGTM.

Have you checked whether we do the right thing with `#pragma pack` on MSVC?


Repository:
  rC Clang

https://reviews.llvm.org/D46218



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D46218: P... Richard Smith - zygoloid via Phabricator via cfe-commits
    • [PATCH] D462... Richard Smith - zygoloid via Phabricator via cfe-commits
    • [PATCH] D462... John McCall via Phabricator via cfe-commits
    • [PATCH] D462... Richard Smith - zygoloid via Phabricator via cfe-commits
    • [PATCH] D462... Richard Smith - zygoloid via Phabricator via cfe-commits
    • [PATCH] D462... Paul Robinson via Phabricator via cfe-commits
    • [PATCH] D462... Richard Smith - zygoloid via Phabricator via cfe-commits

Reply via email to