Skip to content

Commit 6315b49

Browse files
committed
Add warning to the Builder::llvm_config function
1 parent c064521 commit 6315b49

File tree

1 file changed

+4
-0
lines changed
  • src/bootstrap/src/core/builder

1 file changed

+4
-0
lines changed

src/bootstrap/src/core/builder/mod.rs

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1647,6 +1647,10 @@ You have to build a stage1 compiler for `{}` first, and then use it to build a s
16471647
///
16481648
/// Note that this returns `None` if LLVM is disabled, or if we're in a
16491649
/// check build or dry-run, where there's no need to build all of LLVM.
1650+
///
1651+
/// FIXME(@kobzol)
1652+
/// **WARNING**: This actually returns the **HOST** LLVM config, not LLVM config for the given
1653+
/// *target*.
16501654
pub fn llvm_config(&self, target: TargetSelection) -> Option<PathBuf> {
16511655
if self.config.llvm_enabled(target) && self.kind != Kind::Check && !self.config.dry_run() {
16521656
let llvm::LlvmResult { host_llvm_config, .. } = self.ensure(llvm::Llvm { target });

0 commit comments

Comments
 (0)