Skip to content

Enable .ll test suffix for LLVM tests - #5887

Merged
Chris B (llvm-beanz) merged 1 commit into
microsoft:mainfrom
llvm-beanz:cbieneman/enable-llvm-ll-tests
Oct 18, 2023
Merged

Chris B (llvm-beanz) merged 1 commit into
microsoft:mainfrom
llvm-beanz:cbieneman/enable-llvm-ll-tests

Conversation

@llvm-beanz

Copy link
Copy Markdown
Collaborator

This enables .ll as a default test suffix in the LLVM tests. To preserve the low number of unsupported tests, this changes the way we disable tests from using config.unsupported to setting the suffixes to empty.

This enables `.ll` as a default test suffix in the LLVM tests. To
preserve the low number of unsupported tests, this changes the way we
disable tests from using config.unsupported to setting the suffixes to
empty.

@dmpots David Peixotto (dmpots) left a comment

Copy link
Copy Markdown
Contributor

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

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants