simark added a comment.

In, @ilya-biryukov wrote:

> > That looks like another preamble-handling bug. It's much easier to fix and 
> > it's clangd-specific, so I'll make sure to fix that soon.
> >  Thanks for bringing this up, we haven't been testing compile command 
> > changes thoroughly. I'll come up with a fix shortly.
> > 
> > Before this is fixed, you could try writing a similar test with other, 
> > non-preprocessor, flags. See `ClangdVFSTest.ForceReparseCompileCommand` 
> > that tests a similar thing by changing `-xc` to `-xc++`.
> >  I'll make sure to test the `-D` case when fixing the bug.
> Should be fixed by

Thanks, I think I'll simply wait for that fix to get in.

  rCTE Clang Tools Extra

cfe-commits mailing list

Reply via email to