[clang] [clang-repl] Extend the C support. (PR #89804)

[Vassil Vassilev via cfe-commits]

vgvassilev wrote:

I realize I do not entirely understand the role of the IdResolver chain in c++. 
Perhaps we are better of changing this line to:

```cpp
    if (!PP.isIncrementalProcessingEnabled() || getLangOpts().ObjC)
      IdResolver.RemoveDecl(D);
 ```
 to
 ```
    if (!PP.isIncrementalProcessingEnabled() || !getLangOpts().C)
      IdResolver.RemoveDecl(D);
  ```

This way will limit the scope to the actual intent without assuming the 
IdResolver changes are irrelevant for languages such as C++. Can you test if 
that works?

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