search

cpc / openasip

Open Application-Specific Instruction Set processor tools (OpenASIP)
http://openasip.org
Other
140 stars 42 forks source link

Fix warnings that cause tce-selftest to fail under clang #187

Closed sarahec closed 1 year ago

sarahec commented 1 year ago

clang was generating two compile warnings to stderr, causing the self-test to fail.

Fixes:

  1. Removed an assertion that would never fail
  2. Added a default clause to a switch statement that only needed three cases out of 91. Fixes #185
pjaaskel commented 1 year ago

Looks OK, but today I created a 'main' that replaced the old 'master' with the OpenASIP 2.0 stuff pushed in. This will be the new development branch from now on. This branch renamed tce-selftest to oa-selftest. Can you thus rebase to 'main'? Thanks.

sarahec commented 1 year ago

Nice! I'll take care of the rebase.

sarahec commented 1 year ago

The selftest isn't failing anymore under LLVM 15 and openasip-2.0 so I'm going to close this. I'll return to these when cleaning up lints.