[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