Skip to content

Tracking Issue for -Z src-hash-algorithm #70401

Open
@arlosi

Description

@arlosi
Contributor

This is the tracking issue for the unstable option -Z src-hash-algorithm

Steps

  • Adjust documentation
    Stabilization PR

Unresolved Questions

  • Should we have a separate option in the target specification to specify the preferred hash algorithm, or continue to use is_like_msvc?
  • Should continue to have a command line option to override the preferred option? Or is it acceptable to require users to create a custom target specification to override the hash algorithm?

Activity

added
C-tracking-issueCategory: An issue tracking the progress of sth. like the implementation of an RFC
on Mar 25, 2020
added
A-debuginfoArea: Debugging information in compiled programs (DWARF, PDB, etc.)
T-compilerRelevant to the compiler team, which will review and decide on the PR/issue.
B-unstableBlocker: Implemented in the nightly compiler and unstable.
on Mar 25, 2020
added a commit that references this issue on Apr 4, 2020

Auto merge of rust-lang#69718 - arlosi:debughash, r=eddyb

est31

est31 commented on Oct 8, 2020

@est31
Member

#73526 has been merged, updating LLVM to 11 and making it possible to support SHA256 as well. I guess for now one still has to give compatibility for older LLVMs though, so one maybe has to place a few cfg's here and there.

added a commit that references this issue on Nov 1, 2020

Rollup merge of rust-lang#77950 - arlosi:sha256, r=eddyb

3508fa5
added a commit that references this issue on Nov 3, 2020

Rollup merge of rust-lang#77950 - arlosi:sha256, r=eddyb

52405f7
pierwill

pierwill commented on Oct 13, 2021

@pierwill
Member

Are there documentation changes needed for this? I'd be happy to help!

pierwill

pierwill commented on Oct 18, 2021

@pierwill
Member

☝️ @eddyb

arlosi

arlosi commented on Oct 18, 2021

@arlosi
ContributorAuthor

@pierwill It is currently documented as an unstable option here: https://doc.rust-lang.org/beta/unstable-book/compiler-flags/src-hash-algorithm.html

I think the next step here would be to make a stabilization PR and adjust the documentation move it to the stable list of codegen options.

pnkfelix

pnkfelix commented on Sep 9, 2022

@pnkfelix
Member

Visited during T-compiler backlog bonanza

It sounds like this is either ready to stabilize as is (with a shift from a -Z option to something stable), or it should move into something that is specified as part of of the target specification, and we just aren't even sure which of those modes we want this in for the stable form.

Can we maybe get a summary of the current implementation status, as well as the tradeoffs between those two options, to inform that decision?

@rustbot label: S-tracking-needs-summary

added
S-tracking-needs-summaryStatus: It's hard to tell what's been done and what hasn't! Someone should do some investigation.
on Sep 9, 2022
added
A-CLIArea: Command-line interface (CLI) to the compiler
on Mar 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    A-CLIArea: Command-line interface (CLI) to the compilerA-debuginfoArea: Debugging information in compiled programs (DWARF, PDB, etc.)B-unstableBlocker: Implemented in the nightly compiler and unstable.C-tracking-issueCategory: An issue tracking the progress of sth. like the implementation of an RFCS-tracking-design-concernsStatus: There are blocking design concerns.S-tracking-needs-summaryStatus: It's hard to tell what's been done and what hasn't! Someone should do some investigation.T-compilerRelevant to the compiler team, which will review and decide on the PR/issue.

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

      Development

      No branches or pull requests

        Participants

        @pnkfelix@arlosi@jonas-schievink@est31@pierwill

        Issue actions

          Tracking Issue for `-Z src-hash-algorithm` · Issue #70401 · rust-lang/rust