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.

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

Merge r298604 into the 4.0 branch : [SDAG] Fix zeroExtend assertion error #31756

Closed
tstellar opened this issue Mar 24, 2017 · 5 comments
Closed
Assignees
Labels
bugzilla Issues migrated from bugzilla

Comments

@tstellar
Copy link
Collaborator

Bugzilla Link 32409
Resolution FIXED
Resolved on Apr 24, 2017 13:14
Version 4.0
OS All
Blocks #31409

Extended Description

Is this patch OK to merge to the 4.0 branch?

@tstellar
Copy link
Collaborator Author

assigned to @tstellar

@llvmbot
Copy link
Collaborator

llvmbot commented Apr 4, 2017

llvm/llvm-bugzilla-archive#32284 was reopened because of similar cases. If we add this we should also consider: https://reviews.llvm.org/D31625

@bogner
Copy link
Contributor

bogner commented Apr 4, 2017

Change looks good, though it's a bit unfortunate the test is just a "don't crash" test that doesn't actually check anything.

@tstellar
Copy link
Collaborator Author

Merged: r300979

Nirav once D31625 is committed, can you create a separate merge request for that one?

@llvmbot
Copy link
Collaborator

llvmbot commented Apr 24, 2017

Will do.

@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
Labels
bugzilla Issues migrated from bugzilla
Projects
None yet
Development

No branches or pull requests

3 participants