[clang] [clang] Remove dead incremental Parser code (PR #102450)

Ben Barham via cfe-commits cfe-commits at lists.llvm.org
Fri Apr 25 10:02:58 PDT 2025


bnbarham wrote:

> As far as I can tell, -fincremental-extensions should set the language option IncrementalExtensions which in turn is the default for Preprocessor::IncrementalProcessing.

It is true that it is the default, but it's possible for clients to have one without the other. This is the case in Swift in particular (see https://github.com/llvm/llvm-project/pull/65683).

> The few tests we have with -fincremental-extensions pass with this change.

I couldn't think of a great way to test this at the time. If anyone has any ideas though... 😅. Otherwise we could add a comment here to explain why it's needed?

https://github.com/llvm/llvm-project/pull/102450


More information about the cfe-commits mailing list