https://github.com/ArcsinX approved this pull request.
I think this patch makes sense, because `getCorrespondingHeaderOrSource()` is also used in the `DefineOutline` tweak. Without this patch, the priority of extensions is a little strange (cpp=>c=>cc), i.e. without this patch, when we move the definition from the header, if there are `.c` and `.cc` files with the same basename, then priority is given to `.c`, but if there is `.cpp`, then priority is given to `.cpp`. So, the logic as a whole is not very clear. With this patch there is at least a consistent pattern that priority is given to C++ file extensions. https://github.com/llvm/llvm-project/pull/198152 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
