Skip to content

run-make: audit tests that codegen for specific targets #129390

Closed

Description

@jieyouxu

Tests doing codegen for specific targets need //@ needs-llvm-components directives.
These are the tests that are currently missing the directives and failing if LLVM is built without various aarch64/arm/thumb components or wasm.

  • tests\run-make\print-target-list
  • tests\run-make\print-to-output
  • tests\run-make\print-cfg
  • tests\run-make\target-without-atomic-cas

Originally posted by @petrochenkov in #121876 (comment)

Activity

  1. added
    needs-triageThis issue may need triage. Remove when done. See docs forge.rust-lang.org/release/issue-triaging
    on Aug 22, 2024
  2. self-assigned this
    on Aug 22, 2024
  3. added
    A-run-makeArea: port run-make Makefiles to rmake.rs
    A-testsuiteArea: The testsuite used to check the correctness of rustc
    T-compilerRelevant to the compiler team, which will review and decide on the PR/issue.
    C-bugCategory: This is a bug.
    and removed
    needs-triageThis issue may need triage. Remove when done. See docs forge.rust-lang.org/release/issue-triaging
    on Aug 22, 2024
  4. jieyouxu commented on Aug 22, 2024

    @jieyouxu
    MemberAuthor

    E.g. for print-target-list this looks a bit awkward, maybe we can dump //@ needs-llvm-components: $KNOWN_LLVM_COMPONENTS, where $KNOWN_LLVM_COMPONENTS is shamelessly stolen from tidy:

    const KNOWN_LLVM_COMPONENTS: &[&str] = &[
    "aarch64",
    "arm",
    "avr",
    "bpf",
    "csky",
    "hexagon",
    "loongarch",
    "m68k",
    "mips",
    "msp430",
    "nvptx",
    "powerpc",
    "riscv",
    "sparc",
    "systemz",
    "webassembly",
    "x86",
    ];

    Between this directive and COMPILETEST_REQUIRE_ALL_LLVM_COMPONENTS, it is not exactly very clear what's happening.

  5. added 2 commits that reference this issue on Aug 31, 2024
    004014a
    d354d4d
  6. added a commit that references this issue on Aug 31, 2024
    bfb6bb7
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

A-run-makeArea: port run-make Makefiles to rmake.rsA-testsuiteArea: The testsuite used to check the correctness of rustcC-bugCategory: This is a bug.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

    Issue actions