vgvassilev wrote:

> > Oh, we need to adjust 
> > https://github.com/root-project/root/blob/be5d34934de883270683030b3af2cd1195d17ea8/cmake/modules/RootMacros.cmake#L272
> >  to skip in case of C++...
> 
> The link points to an irrelevant project, I assume you mean here 
> https://github.com/llvm/llvm-project/blob/main/clang/lib/Interpreter/IncrementalParser.cpp#L416?
>  If we need to skip for C++, I think we should do it for ObjectiveC as well, 
> like
> 
> ```
> if (ND->getDeclName().getFETokenInfo()  &&
>     !(!PP.isIncrementalProcessingEnabled() || getLangOpts().ObjC || 
> getLangOpts().CPlusPlus)) {
>    ...
> }
> ```

Not sure how that happened. I meant 
https://github.com/llvm/llvm-project/blob/70550cd6aa9f2587e166d6ab9636192af3f3264d/clang/lib/Interpreter/IncrementalParser.cpp#L416-L417

https://github.com/llvm/llvm-project/pull/94471
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to