[clang] [libclang/python] Add tests for equality operators. (PR #138132)
Vlad Serebrennikov via cfe-commits
cfe-commits at lists.llvm.org
Thu May 1 06:57:29 PDT 2025
Endilll wrote:
> @Endilll I added the tests as discussed on #138074 .
Thank you!
> I'm just not sure how/if we can reasonably test the `else` path for `briefComment`: this requires a `libclang.so` built before LLVM 3.2 (the check was added 13 years ago in [b947e82](https://github.com/llvm/llvm-project/commit/b947e8239f311ff88c3f874d120ae4e41c9659d0)). In fact, I'm not entirely sure what the point here is: being able to use the same `libclang.so` with a newer `cindex.py` and then returning nonsense instead of erroring out surely can't be the goal, right? Perhaps we're better off removing the if/else part entirely?
I believe Python bindings are designed to work with the matching libclang version, and we don't have to support quirks of older libclang versions. Users of older versions should use matching bindings version instead.
https://github.com/llvm/llvm-project/pull/138132
More information about the cfe-commits
mailing list