Added comment to explain why only RustIstaller has llvm_tools = false.

rust-lang · Jun 9, 2018 · ab5e3e6 · ab5e3e6
1 parent 4a0e92e
commit ab5e3e6
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/src/bootstrap/tool.rs b/src/bootstrap/tool.rs
@@ -340,6 +340,9 @@ macro_rules! tool {
     }
 }
 
+// FIXME(#51459): We have only checked that RustInstaller does not require
+// the LLVM binaries when running. We should go through all tools to determine
+// if they really need LLVM binaries, and make `llvm_tools` a required argument.
 tool!(
     Rustbook, "src/tools/rustbook", "rustbook", Mode::ToolRustc;
     ErrorIndex, "src/tools/error_index_generator", "error_index_generator", Mode::ToolRustc;