Skip to content

Commit

Permalink
[host] Fix LIT tests when CA_HOST_TARGET_CPU is set
Browse files Browse the repository at this point in the history
The user-defined value was mistakenly passed to
`lit.util.pythonize_bool` which requires it to be a bool or a string
representing on/off: 'on','off','true','false', etc.

Since this is not a boolean option but a string representing a CPU,
there is a simpler safer way of performing this assignment.
  • Loading branch information
frasercrmck committed Jan 3, 2024
1 parent f6b5f2a commit a20c693
Showing 1 changed file with 2 additions and 5 deletions.
7 changes: 2 additions & 5 deletions modules/mux/targets/host/test/lit/lit.site.cfg.in
Original file line number Diff line number Diff line change
Expand Up @@ -34,11 +34,8 @@ lit_config.load_config(config, common_lit_cfg_path)

config.test_exec_root = '@CURRENT_BINARY_DIR@'

config.ca_host_target_cpu = if_enabled(
'@CA_HOST_TARGET_CPU@',
'@CA_HOST_TARGET_CPU@',
None
)
config.ca_host_target_cpu = '@CA_HOST_TARGET_CPU@' or None

config.ca_host_cl_device_name = '@host_CA_HOST_CL_DEVICE_NAME@'

config.ca_host_capabilities = '@host_CAPABILITIES@'
Expand Down

0 comments on commit a20c693

Please sign in to comment.