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 r322003 into 6.0 release #35224

Closed
llvmbot opened this issue Jan 9, 2018 · 3 comments
Closed

Merge r322003 into 6.0 release #35224

llvmbot opened this issue Jan 9, 2018 · 3 comments
Labels
bugzilla Issues migrated from bugzilla

Comments

@llvmbot
Copy link
Collaborator

llvmbot commented Jan 9, 2018

Bugzilla Link 35876
Resolution FIXED
Resolved on Jan 17, 2018 09:48
Version 6.0
OS Linux
Blocks #35152
Reporter LLVM Bugzilla Contributor
CC @zmodem

Extended Description

I'm nominating merging "Teach BaseIndexOffset to correctly handle with indexed operations" for the 6.0.0 branch.

This is a straightforward bugfix in SelectionDAG's address analysis which was triggered by a post-split commit in trunk.
@zmodem
Copy link
Collaborator

zmodem commented Jan 17, 2018

Looking at the r322085 thread, it sounds like there are still problems in this area. But maybe r322003 still makes sense to merge? Please let me know.

@llvmbot
Copy link
Collaborator Author

llvmbot commented Jan 17, 2018

It's still worth merging; the case that r322003 fixes is should be reachable without it.

@zmodem
Copy link
Collaborator

zmodem commented Jan 17, 2018

Merged in r322693.

@zmodem zmodem mentioned this issue Jan 3, 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
@zmodem @llvmbot