Skip to content

CI: gist the disassemblies #54

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

Merged
merged 3 commits into from
Aug 23, 2016
Merged

CI: gist the disassemblies #54

merged 3 commits into from
Aug 23, 2016

Conversation

japaric
Copy link
Member

@japaric japaric commented Aug 22, 2016

closes #53
fixes #51

cc @mattico
crosses fingers

@japaric
Copy link
Member Author

japaric commented Aug 22, 2016

Linux

I can update the docker image to include the openssl header files. Maybe I can even pre-install the gist crate.

OSX

There seems to be a legit error on the osx targets. "API error+"?

Windows

And I'm not sure how to install openssl on appveyor.

@japaric
Copy link
Member Author

japaric commented Aug 22, 2016

openssl

(I hope that someday a production grade ssl library written in Rust will appear so we don't have to rely on openssl anymore)

@japaric
Copy link
Member Author

japaric commented Aug 23, 2016

CALL "C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\vcvarsall.bat"
dumpbin /disasm target/release/librustc_builtins.rlib |
The syntax of the command is incorrect.
Command exited with code 255

Windows shell doesn't like pipes? or may we just need to remove the newline after the pipe.

@japaric
Copy link
Member Author

japaric commented Aug 23, 2016

It passes everything now but OSX is not uploading a gist. I'm going to try something.

but not on OSX because it doesn't work there

closes #53
cc #51
@japaric japaric merged commit 47c5322 into master Aug 23, 2016
@japaric japaric deleted the gist branch August 23, 2016 02:48
tgross35 pushed a commit to tgross35/compiler-builtins that referenced this pull request Feb 23, 2025
tgross35 pushed a commit to tgross35/compiler-builtins that referenced this pull request Feb 23, 2025
77: adding ceilf and floorf  r=japaric a=jackmott

Finishes issues rust-lang#56  and rust-lang#54 

Co-authored-by: Jack Mott <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Upload CI disassembilies to Pastebin/Gist
2 participants