Skip to content
This repository has been archived by the owner on Mar 21, 2024. It is now read-only.

Update broken_constant_eval.pass.cpp #42

Merged
merged 1 commit into from
Oct 7, 2020

Conversation

wmaxey
Copy link
Member

@wmaxey wmaxey commented Oct 7, 2020

This can fix #41
However, I don't know if there is a way to more specifically target a minor gcc version.

This can fix #41
However, I don't know if there is a way to more specifically target a minor gcc version.
@wmaxey wmaxey requested review from brycelelbach and griwes October 7, 2020 06:15
@@ -11,7 +11,7 @@
// UNSUPPORTED: windows, icc, pgi
// XFAIL: clang-9 && c++11
// XFAIL: clang-10 && c++11
// XFAIL: gcc-9 && c++11
// gcc 10.0 is expected to pass, but later versions do not.
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 meant to say GCC 9?

Copy link
Member Author

Choose a reason for hiding this comment

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

See the list in the issue. I might have an inverted sense of what counts as passing because it's an expected failure.

Regardless, 10.0 is not susceptible while 10.1 and 10.2 are.

Copy link
Collaborator

Choose a reason for hiding this comment

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

I see. 10.0 is not a released version, so to avoid confusion we should probably not refer to it.

Copy link
Member Author

@wmaxey wmaxey Oct 7, 2020

Choose a reason for hiding this comment

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

Yeah, the basepoint was just tested to see if I could find exactly when NVCC picked this up. I'll run a full test suite today and commit by afternoon.

Do you want me to edit the comment for clarity?

@wmaxey wmaxey merged commit a6ee80b into main Oct 7, 2020
@wmaxey wmaxey deleted the bugfix/fix-constant-eval-canary branch October 7, 2020 21:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove gcc-9 as expected failure from broken_constant_eval.pass.cpp
2 participants