[PATCH] D19727: [MS] Improved implementation #pragma pack (MS pragmas, part 2)

Denis Zobnin via cfe-commits cfe-commits at lists.llvm.org
Fri Apr 29 09:06:09 PDT 2016


d.zobnin.bugzilla added inline comments.

================
Comment at: lib/Sema/SemaAttr.cpp:277-278
@@ -276,4 @@
-      // empty.
-      Diag(PragmaLoc, diag::warn_pragma_pop_failed)
-          << "pack" << (Name ? "no record matching name" : "stack empty");
-
----------------
Here I removed the "no record matching name" diagnostics, which wasn't covered by any test.
I'm going to rework and restore it in future patch.


http://reviews.llvm.org/D19727





More information about the cfe-commits mailing list