Jinjie-Huang wrote:

Thanks for the review. 
> While I know this is disabled by default, have we considered the compilation 
> time impact when it is enabled?

Regarding the execution time, I tested with some internal files (ranging from 
average to small inputs) and didn't see any significant difference:
```
File No.        Baseline(s)  With Diagnostic(s)    Gap
1               58.647           59.69                 1.78%
2               42.968           43.437                1.09%
3               20.14            20.195                0.27%
4               5.057            5.057                 -0.02%
5               0.031            0.031                 0.00%
```


https://github.com/llvm/llvm-project/pull/162491
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to