[all-commits] [llvm/llvm-project] 91285e: [clang-reorder-fields] Handle macros fields declar...

Clement Courbet via All-commits all-commits at lists.llvm.org
Thu Nov 28 23:36:44 PST 2024


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 91285e26bf67179a6804c931203c93f07c08183d
      https://github.com/llvm/llvm-project/commit/91285e26bf67179a6804c931203c93f07c08183d
  Author: Clement Courbet <courbet at google.com>
  Date:   2024-11-29 (Fri, 29 Nov 2024)

  Changed paths:
    M clang-tools-extra/clang-reorder-fields/ReorderFieldsAction.cpp
    A clang-tools-extra/test/clang-reorder-fields/FieldAnnotationsInMacros.cpp

  Log Message:
  -----------
  [clang-reorder-fields] Handle macros fields declarations. (#118005)

Right now fields with macro declarations break the tool:

```

struct Foo {
  Mutex mu;
  int x GUARDED_BY(mu);
  int y;
};
```

reordered by mu,y,x yields:

```
struct Foo {
  Mutex mu;
  int y GUARDED_BY(mu);
  int x;
};
```



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


More information about the All-commits mailing list