mydeveloperday wrote:

While I get this fixes the guessLanguage limitation, if its OOM the I guess 
isn't it going to be OOM for any format? (as you mention above), couldn't we 
just use the .clang-format-ignore file to say... hey don't bother with this 
file, we can't do it...

I feel like this will be a piece of code if we accept, might likely never go 
beyond the guess language usage. So it will just become more technical debt. 
Fixing the underlying problem isn't easy because it really amounts to changing 
the whole way clang-format deals with preprocessor paths.


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

Reply via email to