Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[builtins] Guard the divtc3_test.c test with CRT_HAS_TF_MODE #71876

Merged
merged 1 commit into from
Nov 10, 2023

Conversation

aemerson
Copy link
Contributor

@aemerson aemerson commented Nov 9, 2023

The bots on https://green.lab.llvm.org/green/job/clang-stage1-RA/ were
failing this test.

Copy link
Member

@arichardson arichardson left a comment

Choose a reason for hiding this comment

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

I am very surprised the librt_has_divtc3 feature is being defined. It sounds like something is going wrong there? Adding the guard looks good to me but we should really figure out why that feature is incorrectly set.

@aemerson
Copy link
Contributor Author

I think it's something to do with the clang version on the bot being older by a few years. I'm not entirely sure, can this go in anyway since we need to get it back to green?

@arichardson
Copy link
Member

I think it's something to do with the clang version on the bot being older by a few years. I'm not entirely sure, can this go in anyway since we need to get it back to green?

I'm happy for this to be merged, but could you file an issue that the feature detection is broken?

@aemerson
Copy link
Contributor Author

#71971

@aemerson aemerson merged commit 77d75dc into llvm:main Nov 10, 2023
4 checks passed
@aemerson aemerson deleted the divtc3_test_guard branch November 10, 2023 19:30
@rorth
Copy link
Collaborator

rorth commented Nov 13, 2023

This patch broke the Solaris/sparcv9 bot, which unfortunately was completely ignored:

XPASS: Builtins-sparc-sunos:: divtc3_test.c

Issue #71971 lacks detail to decide if the workaround patch is correct. I suspect one could simply remove the XFAIL, but that might well be just another hack.

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

Successfully merging this pull request may close these issues.

4 participants