[PATCH] D35972: Add warning to clang-reorder-fields when reordering breaks member init list dependencies

Sam Conrad via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Jul 29 18:56:36 PDT 2017


sameconrad updated this revision to Diff 108812.
sameconrad added a comment.

Add additional tests for cases with base and derived classes.
Add clarifying comments about emitting warnings for base class members during reordering checks.
Add fix for bug where reordering in init lists that contain non-member initializers causes segfault.


https://reviews.llvm.org/D35972

Files:
  clang-reorder-fields/ReorderFieldsAction.cpp
  test/clang-reorder-fields/ClassDerived.cpp
  test/clang-reorder-fields/FieldDependencyWarning.cpp
  test/clang-reorder-fields/FieldDependencyWarningDerived.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D35972.108812.patch
Type: text/x-patch
Size: 7306 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20170730/8bf63215/attachment.bin>


More information about the cfe-commits mailing list