Skip to content

Conversation

@farzonl
Copy link
Member

@farzonl farzonl commented Jan 28, 2026

  • Fix tests missing 16 bit requires
  • Fix Missing XFAILS for older AMD GPUs that were just called Radeon

test/lit.cfg.py Outdated
config.available_features.add("NV")
if "AMD" in device["Description"]:
if ("AMD" in device["Description"] or
"Radeon" in device["Description"]):
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One of the AMD GPUs I test with via egpu Has a device name of Radeon RX 580 Series. without this change the XFAILS for AMD are never applied and we get over 10 extra unnecessary test failures.

- Fix tests missing 16 bit requires
- Fix Missing XFAILS for older AMD GPUs that were just called Radeon
# Bug https://github.com/llvm/offload-test-suite/issues/393
# XFAIL: Metal

# REQUIRES: Float16
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is this the same as saying Requires Half? that is what the other tests do i think?

Copy link

@tex3d tex3d left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants