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

Merge r343369 into the 7.0 branch #39100

Closed
zmodem opened this issue Nov 22, 2018 · 2 comments
Closed

Merge r343369 into the 7.0 branch #39100

zmodem opened this issue Nov 22, 2018 · 2 comments
Labels
bugzilla Issues migrated from bugzilla

Comments

@zmodem
Copy link
Collaborator

zmodem commented Nov 22, 2018

Bugzilla Link 39753
Resolution FIXED
Resolved on Dec 05, 2018 19:16
Version 7.0
OS Linux
Blocks #38454
CC @tstellar
Fixed by commit(s) r343369 r348450 r348451 r348452 r348453

Extended Description

r343369 "[cxx2a] Fix warning triggered by r343285"

This is a warning fix without which, compiling LLVM with recent Clang versions causes warnings.

Not critical, but it would be nice to get in. I think it's ABI-safe, since it's removing functions that couldn't be called before as they were implicitly deleted.

(Note that the commit spans multiple modules in the repository.)
@tstellar
Copy link
Collaborator

tstellar commented Dec 5, 2018

https://reviews.llvm.org/rL343369

@tstellar
Copy link
Collaborator

tstellar commented Dec 6, 2018

Merged:

llvm: r348450
clang: r348451
clang-tools-extra: r348452
libcxx: r348453

@tstellar tstellar mentioned this issue Sep 27, 2018
Closed
@llvmbot llvmbot transferred this issue from llvm/llvm-bugzilla-archive Dec 10, 2021
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bugzilla Issues migrated from bugzilla
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@tstellar @zmodem