[cfe-dev] libclang support for Matchers

[Manuel Klimek via cfe-dev]

On Thu, Nov 30, 2017, 8:13 PM Jonathan Coe <jonathanbcoe at gmail.com> wrote:

>
>
> On 30 Nov 2017, at 16:20, Manuel Klimek <klimek at google.com> wrote:
>
> On Thu, Nov 30, 2017 at 5:17 PM Jonathan Coe <jonathanbcoe at gmail.com>
> wrote:
>
>> Does the marchers API offer the stability that libclang tries to provide?
>> If not, exposing marchers will be tough
>>
>
> As the dynamic matchers use strings, what stability concerns do you have?
>
>
>
> Will the supported strings be stable?
>

Reasonably so, but not libclang C API style backwards compatible forever.
The question is how much that matters to libclang users (I honestly have no
idea). C API stability seems to be obviously more important, as it can just
crash a program. The worst thing with a wrong string here is that you'll
get an error back.


>
>> On 30 Nov 2017, at 11:05, Manuel Klimek via cfe-dev <
>> cfe-dev at lists.llvm.org> wrote:
>>
>> On Thu, Nov 30, 2017 at 5:56 AM James Mitchell via cfe-dev <
>> cfe-dev at lists.llvm.org> wrote:
>>
>>> I'm looking to add support for matchers using the "C" API and Python
>>> bindings.
>>>
>>> I have started doing some initial work to add support for this:
>>>
>>> https://github.com/ReDucTor/clang/commit/d367e9ab34668da6341307765c303fbbd1b673e3
>>>
>>> It would be good to get any advice/suggestions relating to these changes
>>> I'm still getting familiar with the related APIs and coding standards.
>>>
>>
>> If you find it useful, it seems like a small directed patch that makes
>> sense :)
>>
>>
>>> _______________________________________________
>>> cfe-dev mailing list
>>> cfe-dev at lists.llvm.org
>>> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev
>>>
>> _______________________________________________
>> cfe-dev mailing list
>> cfe-dev at lists.llvm.org
>> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev
>>
>>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20171201/8e3cd095/attachment.html>