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 r344264 into the 7.0 branch : [COFF] Set proper pointer size alignment for LocalImportChunk #38661

Closed
mstorsjo opened this issue Oct 16, 2018 · 7 comments
Closed

Merge r344264 into the 7.0 branch : [COFF] Set proper pointer size alignment for LocalImportChunk #38661

mstorsjo opened this issue Oct 16, 2018 · 7 comments
Assignees
@rui314
Labels
bugzilla Issues migrated from bugzilla

Comments

@mstorsjo
Copy link
Member

Bugzilla Link 39313
Resolution FIXED
Resolved on Oct 22, 2018 09:53
Version 7.0
OS All
Blocks #38454
CC @rui314,@tstellar
Fixed by commit(s) r344264 r344920

Extended Description

Is it OK to merge the following revision(s) to the 7.0 branch?
@mstorsjo
Copy link
Member Author

assigned to @rui314

@mstorsjo
Copy link
Member Author

https://reviews.llvm.org/rL344264

@tstellar
Copy link
Collaborator

Hi Rui,

Is this OK to merge?

@rui314
Copy link
Member

rui314 commented Oct 22, 2018

Yes.

@tstellar
Copy link
Collaborator

Merged: r342187

@tstellar
Copy link
Collaborator

Merged: r342187

This is the wrong commit.

@tstellar
Copy link
Collaborator

Merged: r344920

@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

@rui314 rui314

Labels
bugzilla Issues migrated from bugzilla
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@rui314 @mstorsjo @tstellar