bcraig added inline comments.

================
Comment at: test/Analysis/padding_cpp.cpp:204-212
+// expected-warning@+1{{Excessive padding in 'struct FakeIntSandwich'}}
+struct FakeIntSandwich {
+  char c1;
+  int i;
+  char c2;
+};
+
----------------
Looking at this again... what new cases does this catch?  FakeIntSandwich was 
caught before (it is identical to 'PaddedA', and AnotherIntSandwich generated 
no warning before.  So what is different?

EBO1 and EBO2 are cases above that would be nice to catch, but aren't being 
caught right now.


Repository:
  rC Clang

https://reviews.llvm.org/D53206



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to