Skip to content

Commit

Permalink
[ExecutionEngine][test][RISCV] Don't mark RISCV as unsupported
Browse files Browse the repository at this point in the history
Although MCJIT is unsupported, we can and should be running the other
tests. Stacks on top of llvm#124463 (needed to avoid unsupported MCJIT tests
in the top-level test/ExecutionEngine directory running when they
shouldn't).

This effectively reverts b8feeba.

ninja check-llvm-executionengine before:
Total Discovered Tests: 335
  Unsupported: 335 (100.00%)

ninja check-llvm-executionengine after:
Total Discovered Tests: 335
  Unsupported      : 125 (37.31%)
  Passed           : 209 (62.39%)
  Expectedly Failed:   1 (0.30%)
  • Loading branch information
asb committed Jan 26, 2025
1 parent e463e95 commit cb71a00
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion llvm/test/ExecutionEngine/lit.local.cfg
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
if config.root.native_target in ['Sparc', 'SystemZ', 'Hexagon', 'RISCV']:
if config.root.native_target in ['Sparc', 'SystemZ', 'Hexagon']:
config.unsupported = True

# ExecutionEngine tests are not expected to pass in a cross-compilation setup.
Expand Down

0 comments on commit cb71a00

Please sign in to comment.