search

checkedc / checkedc-llvm-project

This was a fork of Checked C clang used from 2021-2024. The changes have been merged into the original Checked C clang repo, which is now at https://github.com/checkedc/checkedc-clang.
https://www.checkedc.org
13 stars 19 forks source link

Fix additional failing test cases #1235

Closed dtarditi closed 1 week ago

dtarditi commented 1 week ago

This change fixes additional failing test cases for the upgrade to clang 17 sources. It also includes changes for the move of the checkedc-clang repo to the Checked C organization.

The changes include:

  1. Use the driver marshalling infrastructure for setting various flags for dumping Checked C internal compiler information.
  2. Fix some bad merges of test cases. The bad merges happened merging the Checked C changes into the clang 17 sources.
  3. Fix some test cases that broke because of a change in the formatting of array types. There is no longer a space between the declarator name the dimension of the array.
  4. Clang now produces some additional warnings and errors, some of which were conditions that Checked C was already complaining about.

This significantly reduce the number of failing test cases for the clang-17 upgrade. For a Window x64 debug build,ninja checkedc-clang produces the following results:

  Skipped          :    30
  Unsupported      :   237
  Passed           : 33364
  Expectedly Failed:    39
  Failed           :    68
dtarditi commented 1 week ago

These changes should be merge to the 17-init-main branch on the checkedc-clang repo.