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

[REG2.068a] Issue 14814 - ld: GOT load reloc does not point to a movq instruction #4828

Closed
wants to merge 1 commit into from
Closed

[REG2.068a] Issue 14814 - ld: GOT load reloc does not point to a movq instruction #4828

wants to merge 1 commit into from

Conversation

9rnsr
Copy link
Contributor

@9rnsr 9rnsr commented Jul 22, 2015

@9rnsr
Copy link
Contributor Author

9rnsr commented Jul 22, 2015

I'm expecting to fail the test code in OSX...

@9rnsr
Copy link
Contributor Author

9rnsr commented Jul 22, 2015

OK, got it.
https://auto-tester.puremagic.com/show-run.ghtml?projectid=1&runid=1659627&isPull=true

 ... runnable/link14814.d           (-inline -release -g -O -fPIC)
Test failed.  The logged output:
../src/dmd -conf= -m64 -Irunnable   -odgenerated/runnable -c runnable/link14814.d
../src/dmd -conf= -m64 -Irunnable   -odgenerated/runnable -c runnable/imports/link14814a.d
../src/dmd -conf= -m64   -odgenerated/runnable -ofgenerated/runnable/link14814_0 generated/runnable/link14814.o generated/runnable/link14814a.o
ld: GOT load reloc does not point to a movq instruction in '_D7imports10link14814a4fun4FZv' from generated/runnable/link14814a.o for architecture x86_64
collect2: ld returned 1 exit status
--- errorlevel 1

I updated to apply my patch. Now I'm expecting all green.

@9rnsr 9rnsr changed the title [WIP][REG2.068a] Issue 14814 - ld: GOT load reloc does not point to a movq instruction [REG2.068a] Issue 14814 - ld: GOT load reloc does not point to a movq instruction Jul 22, 2015
@9rnsr
Copy link
Contributor Author

9rnsr commented Jul 22, 2015

All green, ready to merge.

@9rnsr 9rnsr mentioned this pull request Jul 22, 2015
Closed
@MartinNowak
Copy link
Member

Thanks, needs to go into stable though.

@MartinNowak MartinNowak added this to the 2.068 milestone Jul 22, 2015
@9rnsr 9rnsr closed this Jul 23, 2015
@9rnsr 9rnsr mentioned this pull request Jul 23, 2015
Merged
@9rnsr
Copy link
Contributor Author

9rnsr commented Jul 23, 2015

See #4829.

@WalterBright
Copy link
Member

Thanks Kenji, Martin and Timothee.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
2.068
3 participants
@9rnsr @MartinNowak @WalterBright