This is an archive of the discontinued LLVM Phabricator instance.

Make check-compiler-rt subtarget respect LLVM_LIT_TOOLS_DIR
Needs ReviewPublic

Authored by gbedwell on Nov 27 2017, 2:15 PM.

Details

Summary

We ran into some strange test failures on a Windows machine that didn't have *nix style commands on the path, but was setting a path to the gnuwin32 tools via LLVM_LIT_TOOLS_DIR. Both the check-compiler-rt and check-lit tests weren't taking notice of this option.

This patch takes care of the compiler-rt part (see D40520 for the check-lit part).

Diff Detail

Event Timeline

gbedwell created this revision.Nov 27 2017, 2:15 PM
espindola edited reviewers, added: espindola; removed: rafael.Mar 15 2018, 8:42 AM

Delayed ping. Apologies for the delay. I started paternity leave shortly after starting this review, and never got a chance to come back to it until now.