Skip to content

Conditional compilation for native items #610

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

Closed
brson opened this issue Jul 1, 2011 · 0 comments
Closed

Conditional compilation for native items #610

brson opened this issue Jul 1, 2011 · 0 comments
Assignees
Labels
A-frontend Area: Compiler frontend (errors, parsing and HIR) E-easy Call for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue.

Comments

@brson
Copy link
Contributor

brson commented Jul 1, 2011

Pending completion of #609, native items should support conditional compilation with the #[cfg(...)] attribute.

@ghost ghost assigned brson Jul 1, 2011
@brson brson closed this as completed in b79ea48 Jul 5, 2011
keeperofdakeys pushed a commit to keeperofdakeys/rust that referenced this issue Dec 12, 2017
Rebase of rust-lang#610 and also move emscripten up much higher in the hierarchy to
ensure that it doesn't have too much of a ripple effect on other platforms.

This involved moving down a good number of definitions, but hopefully was done
with care to not break anything!
keeperofdakeys pushed a commit to keeperofdakeys/rust that referenced this issue Dec 12, 2017

Partially verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
We cannot verify signatures from co-authors, and some of the co-authors attributed to this commit require their commits to be signed.
Add asmjs/wasm32 to CI

Rebase of rust-lang#610 and also move emscripten up much higher in the hierarchy to
ensure that it doesn't have too much of a ripple effect on other platforms.

This involved moving down a good number of definitions, but hopefully was done
with care to not break anything!
ZuseZ4 pushed a commit to EnzymeAD/rust that referenced this issue Mar 7, 2023

Unverified

This commit is not signed, but one or more authors requires that any commit attributed to them is signed.
* Fix phi predecessor removal if given placeholder

* Fix LLVM 7
celinval added a commit to celinval/rust-dev that referenced this issue Jun 4, 2024

Unverified

This commit is not signed, but one or more authors requires that any commit attributed to them is signed.
…#610)

During codegen of function calls, we were ignoring argument of Unit
type. This was causing a failure due to missing arguments when an
intrinsic or closure was being called with an '()' as an argument.

This change modifies how we deal with Unit types during function
definition and function call.
GuillaumeGomez pushed a commit to GuillaumeGomez/rust that referenced this issue Apr 18, 2025

Unverified

This commit is not signed, but one or more authors requires that any commit attributed to them is signed.
Update repositories URL for repositories moved to `rust-lang` organization
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-frontend Area: Compiler frontend (errors, parsing and HIR) E-easy Call for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue.
Projects
None yet
Development

No branches or pull requests

1 participant