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.

Sign up for GitHub

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

Jump to bottom

New instances of -Wbuiltin-macro-redefined after LLVM commit 0123deb3a6f0 #1855

Closed
nathanchance opened this issue May 26, 2023 · 3 comments
Closed

New instances of -Wbuiltin-macro-redefined after LLVM commit 0123deb3a6f0 #1855

nathanchance opened this issue May 26, 2023 · 3 comments
Labels
[BUG] llvm (main) A bug in an unreleased version of LLVM (this label is appropriate for regressions) [FIXED][LLVM] main This bug was only present and fixed in an unreleased version of LLVM Reported upstream This bug was filed on LLVM’s issue tracker, Phabricator, or the kernel mailing list.

Comments

@nathanchance
Copy link
Member

llvm/llvm-project@0123deb

I have started an upstream discussion around this: https://reviews.llvm.org/D144654#4376428
@nathanchance nathanchance added [BUG] llvm (main) A bug in an unreleased version of LLVM (this label is appropriate for regressions) Reported upstream This bug was filed on LLVM’s issue tracker, Phabricator, or the kernel mailing list. labels May 26, 2023
@nickdesaulniers
Copy link
Member

https://reviews.llvm.org/D151741 might fix this?

@nickdesaulniers nickdesaulniers added the [PATCH] Submitted A patch has been submitted for review label May 30, 2023
This was referenced May 30, 2023
Closed
Closed
@nathanchance
Copy link
Member Author

I am currently testing D151741 and the ARCH=arm warnings have already disappeared, so I do expect that change to resolve this issue.

@nickdesaulniers
Copy link
Member

Fixed: https://reviews.llvm.org/rG844e9534c6d99ddb6bada740839760fa24d17cb6

@nickdesaulniers nickdesaulniers added [FIXED][LLVM] main This bug was only present and fixed in an unreleased version of LLVM and removed [PATCH] Submitted A patch has been submitted for review labels Jun 1, 2023
@nathanchance nathanchance mentioned this issue Jun 1, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
[BUG] llvm (main) A bug in an unreleased version of LLVM (this label is appropriate for regressions) [FIXED][LLVM] main This bug was only present and fixed in an unreleased version of LLVM Reported upstream This bug was filed on LLVM’s issue tracker, Phabricator, or the kernel mailing list.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@nickdesaulniers @nathanchance