[PATCH] D34114: [clang] Change the condition of unnecessary packed warning

Chih-Hung Hsieh via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jul 26 16:24:04 PDT 2017


chh added inline comments.


================
Comment at: lib/AST/RecordLayoutBuilder.cpp:1888
-    if (Packed && UnpackedAlignment > CharUnits::One() && 
-        getSize() == UnpackedSize)
       Diag(D->getLocation(), diag::warn_unnecessary_packed)
----------------
Why not keeping the (getSize() == UnpackedSize) condition?



https://reviews.llvm.org/D34114





More information about the cfe-commits mailing list