Skip to content

Adding a doc comment talking about performance impacts of assume #77561

Closed
@tesuji

Description

@tesuji

So LLVM hasn't changed much since nagisa wrote #54995 (comment).
Here the link to related regression caused by using constly assume intrinsic in const fn: #77023 (comment)

I think we could include a warning for assume intrinsic about using it in const fn and should run a perf for PR using it.

/// This intrinsic does not have a stable counterpart.
#[rustc_const_unstable(feature = "const_assume", issue = "76972")]
pub fn assume(b: bool);

cc @oli-obk

Metadata

Metadata

Assignees

No one assigned

    Labels

    A-LLVMArea: Code generation parts specific to LLVM. Both correctness bugs and optimization-related issues.A-docsArea: Documentation for any part of the project, including the compiler, standard library, and toolsA-intrinsicsArea: IntrinsicsT-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

    Issue actions