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

Fix LLVMRustInlineAsmVerify return type mismatch #55128

Merged
merged 1 commit into from
Oct 18, 2018

Conversation

varkor
Copy link
Member

@varkor varkor commented Oct 16, 2018

Fixes #54918.

r? @rkruppe
cc @levex

@varkor varkor force-pushed the LLVMRustInlineAsmVerify-return-bool branch from 37de0d2 to f40932f Compare October 16, 2018 17:22
@hanna-kruppe
Copy link
Contributor

@bors r+

@bors
Copy link
Collaborator

bors commented Oct 16, 2018

📌 Commit f40932f has been approved by rkruppe

@bors bors added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Oct 16, 2018
@levex
Copy link
Contributor

levex commented Oct 16, 2018

Nice catch @varkor, thank you!

kennytm added a commit to kennytm/rust that referenced this pull request Oct 18, 2018
…rn-bool, r=rkruppe

Fix LLVMRustInlineAsmVerify return type mismatch

Fixes rust-lang#54918.

r? @rkruppe
cc @levex
bors added a commit that referenced this pull request Oct 18, 2018
Rollup of 18 pull requests

Successful merges:

 - #54646 (improve documentation on std::thread::sleep)
 - #54933 (Cleanup the rest of codegen_llvm)
 - #54964 (Run both lldb and gdb tests)
 - #55016 (Deduplicate some code and compile-time values around vtables)
 - #55031 (Improve verify_llvm_ir config option)
 - #55050 (doc std::fmt: the Python inspiration is already mentioned in precedin…)
 - #55077 (rustdoc: Use dyn keyword when rendering dynamic traits)
 - #55080 (Detect if access to localStorage is forbidden by the user's browser)
 - #55090 (regression test for move out of borrow via pattern)
 - #55102 (resolve: Do not skip extern prelude during speculative resolution)
 - #55104 (Add test for #34229)
 - #55111 ([Rustc Book] Explain --cfg's arguments)
 - #55122 (Cleanup mir/borrowck)
 - #55127 (Remove HybridBitSet::dummy)
 - #55128 (Fix LLVMRustInlineAsmVerify return type mismatch)
 - #55142 (miri: layout should not affect CTFE checks (outside of validation))
 - #55151 (Cleanup nll)
 - #55161 ([librustdoc] Disable spellcheck for search field)
@bors bors merged commit f40932f into rust-lang:master Oct 18, 2018
@varkor varkor deleted the LLVMRustInlineAsmVerify-return-bool branch October 18, 2018 10:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants