ian-twilightcoder wrote:

> From Jan:
> 
> > Maybe we can add the number of external HeaderFileInfos to 
> > HeaderSearch::PrintStats() and have a test that checks for it.
> > The flag to enable that output is -show-stats I think.

We tried this, but it seems that by the time the modules compile, everything 
ends up being local instead of external anyway. I think a unit test is probably 
good enough for this one since we aren't even sure what the practical 
consequence is of this bug.

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

Reply via email to