https://github.com/NagyDonat commented:

Thanks for the updates!

I have a few new suggestions in inline comments.

I also have one organizational suggestion for testing: as this area requires 
many separate test files (because there may be only one `main` function per 
file), you should place all the new tests in a freshly created subdirectory of 
`test/Analysis` (e.g. `test/Analysis/taint-main-parameters` or something 
similar).

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

Reply via email to