Skip to content

Fix AST lit failures. #13557

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

Merged
merged 1 commit into from
Apr 25, 2024
Merged

Fix AST lit failures. #13557

merged 1 commit into from
Apr 25, 2024

Conversation

haonanya
Copy link
Contributor

@haonanya haonanya commented Apr 25, 2024

Let MathErrno reference the previous item FPAccuracy.
Fix:
Clang::AST/ast-dump-fpfeatures.cpp
Clang::AST/ast-dump-fpfeatures.m
Clang::AST/ast-dump-late-parsing.cpp

Let MathErrno reference the previous item FPAccuracy.
@haonanya haonanya requested a review from a team as a code owner April 25, 2024 02:20
@haonanya haonanya requested a review from mikerice1969 April 25, 2024 02:21
@smanna12 smanna12 requested a review from zahiraam April 25, 2024 03:37
@smanna12
Copy link
Contributor

smanna12 commented Apr 25, 2024

This seems due to community patch: https://github.com/intel-restricted/applications.compilers.llvm-project/commit//a04624206ddf03dc54d5c372e7eac13575b4124b

@zahiraam could you please take a look at? Thanks

Copy link
Contributor

@zahiraam zahiraam left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks.

@jsji jsji merged commit 49910d4 into sycl-web Apr 25, 2024
jsji pushed a commit that referenced this pull request Apr 25, 2024
Let MathErrno reference the previous item FPAccuracy.
Fix:
Clang::AST/ast-dump-fpfeatures.cpp
Clang::AST/ast-dump-fpfeatures.m
Clang::AST/ast-dump-late-parsing.cpp
@bader bader deleted the CMPLRLLVM-57954 branch May 14, 2024 15:58
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.

6 participants