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

Distribute a statically linked snapshot binary #10809

Merged
merged 1 commit into from
Dec 7, 2013

Conversation

alexcrichton
Copy link
Member

Now that we have the necessary logic in rustc for windows, this is possible to land.

@brson
Copy link
Contributor

brson commented Dec 5, 2013

I'd prefer to move all llvm code into its own llvm crate, but this is fine as an intermediate step.

In order to keep up to date with changes to the libraries that `llvm-config`
spits out, the dependencies to the LLVM are a dynamically generated rust file.
This file is now automatically updated whenever LLVM is updated to get kept
up-to-date.

At the same time, this cleans out some old cruft which isn't necessary in the
makefiles in terms of dependencies.

Closes rust-lang#10745
Closes rust-lang#10744
bors added a commit that referenced this pull request Dec 7, 2013
Now that we have the necessary logic in rustc for windows, this is possible to land.
@bors bors closed this Dec 7, 2013
@bors bors merged commit e91ffb0 into rust-lang:master Dec 7, 2013
@alexcrichton alexcrichton deleted the static-snapshot branch December 7, 2013 18:16
flip1995 pushed a commit to flip1995/rust that referenced this pull request Jun 2, 2023
Fix missing block for unsafe code

If a block is declared as unsafe, it needs an extra layer of curly braces around it.

Fixes rust-lang#10808

This code adds handling for `UnsafeSource::UserProvided` block, i.e. `unsafe { ... }`. Note that we do not handle the `UnsafeSource::CompilerGenerated` as it seems to not be possible to generate that with the user code (?), or at least doesn't seem to be needed to be handled explicitly.

There is an issue with this code: it does not add an extra indentation for the unsafe blocks. I think this is a relatively minor concern for such an edge case, and should probably be done by a separate PR (fixing compile bug is more important than getting styling perfect especially when `rustfmt` will fix it anyway)

```rust
// original code
unsafe {
  ...
}

// code that is now generated by this PR
{ unsafe {
  ...
} }

// what we would ideally like to get
{
  unsafe {
    ...
  }
}
```

changelog: [`single_match`](https://rust-lang.github.io/rust-clippy/master/#single_match): Fix suggestion for `unsafe` blocks
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.

3 participants