Closed secure-sw-dev-bot closed 2 years ago
Comment from @mgrang:
I see changes in several files because your PR does not seem to be on latest master. Could you please merge master into your PR? This would make it easier to review the changes.
Comment from @Machiry:
I see changes in several files because your PR does not seem to be on latest master. Could you please merge master into your PR? This would make it easier to review the changes.
Actually, they are on top of the master. The huge number of files is because of the new tests (318) we added.
Comment from @mgrang:
Sorry, I am having difficulty reviewing this PR effectively because of its size. Is it possible to break it down into smaller PRs?
This issue was copied from https://github.com/microsoft/checkedc-clang/issues/878
This pull request includes a lot of changes to cconv tool.
We fixed various issues, added a lot (318) of test cases, improved the reliability and accuracy of the tool done over past 2 months.