Skip to content

Commit 6c24eee

Browse files
authored
Rollup merge of #147280 - workingjubilee:you-can-put-your-gcc-in-my-llvm-components, r=Kobzol
Return to needs-llvm-components being info-only Partially revert a535042 Even with non-LLVM codegen backends, we want to allow for annotations that express dependencies to LLVM-specific parts of the test suite. This includes `//@ needs-llvm-components`, which just allows checking that LLVM is built with relevant target support before the test is run. It does not assert the test cannot work with another codegen backend.
2 parents dade9fe + 99550fb commit 6c24eee

File tree

1 file changed

+1
-4
lines changed
  • src/tools/compiletest/src/directives

1 file changed

+1
-4
lines changed

src/tools/compiletest/src/directives/needs.rs

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -281,10 +281,7 @@ pub(super) fn handle_needs(
281281

282282
// Handled elsewhere.
283283
if name == "needs-llvm-components" {
284-
if config.default_codegen_backend.is_llvm() {
285-
return IgnoreDecision::Continue;
286-
}
287-
return IgnoreDecision::Ignore { reason: "LLVM specific test".into() };
284+
return IgnoreDecision::Continue;
288285
}
289286

290287
let mut found_valid = false;

0 commit comments

Comments
 (0)