[all-commits] [llvm/llvm-project] a17b5b: [clang-reorder-fields] Prevent rewriting unsupport...

[Vladimir Vuksanovic via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: a17b5bce8c9bfa7767827df392855cff9f2af372
      https://github.com/llvm/llvm-project/commit/a17b5bce8c9bfa7767827df392855cff9f2af372
  Author: Vladimir Vuksanovic <109677816+vvuksanovic at users.noreply.github.com>
  Date:   2025-06-22 (Sun, 22 Jun 2025)

  Changed paths:
    M clang-tools-extra/clang-reorder-fields/ReorderFieldsAction.cpp
    A clang-tools-extra/test/clang-reorder-fields/MacroExpandsToMultipleFields.cpp
    A clang-tools-extra/test/clang-reorder-fields/MultipleFieldDeclsInStatement.cpp
    A clang-tools-extra/test/clang-reorder-fields/PreprocessorDirectiveAroundDefinition.cpp
    A clang-tools-extra/test/clang-reorder-fields/PreprocessorDirectiveAroundFields.cpp
    A clang-tools-extra/test/clang-reorder-fields/PreprocessorDirectiveInDefinition.cpp

  Log Message:
  -----------
  [clang-reorder-fields] Prevent rewriting unsupported cases (#142149)

Add checks to prevent rewriting when doing so might result in incorrect
code. The following cases are checked:
- There are multiple field declarations in one statement like `int a, b`
- Multiple fields are created from a single macro expansion
- Preprocessor directives are present in the struct



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications