Michael137 wrote: > I can not check what caused exactly the problems but the change looks correct > and in change D154764 there seems to be no reason for re-ordering at > non-record objects. As an improvement, some tests could be added that check > for re-ordering at `RecordDecl` and (this is the more important) no > re-ordering at `ObjCInterfaceDecl`.
Agreed it would be nice to get a test for this. We have a test that checks re-ordering of `RecordDecl`s (`CXXRecordDeclFieldOrderShouldNotDependOnImportOrder`). For `ObjCInterfaceDecl`s it's trickier because, AFAIK, they neither support in-class member initialisers nor function bodies that could trigger a different import order. Any ideas? https://github.com/llvm/llvm-project/pull/77079 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
