This is an archive of the discontinued LLVM Phabricator instance.

Use full path to clang.exe in LLDB_TEST_COMPILER
Needs RevisionPublic

Authored by zturner on May 13 2016, 10:14 AM.

Details

Reviewers
gkistanova
Summary

I basically just copied your code, so this should be correct, but just want to make sure I'm not missing anything.

Diff Detail

Event Timeline

zturner updated this revision to Diff 57211.May 13 2016, 10:14 AM
zturner retitled this revision from to Use full path to clang.exe in LLDB_TEST_COMPILER.
zturner updated this object.
zturner added a reviewer: gkistanova.
zturner added a subscriber: llvm-commits.
gkistanova accepted this revision.May 13 2016, 3:06 PM
gkistanova edited edge metadata.

Looks fine.
The only possible issue is the special symbols termination in paths. You may want to add quotation marks around paths or something.

This revision is now accepted and ready to land.May 13 2016, 3:06 PM
gkistanova requested changes to this revision.May 13 2016, 3:17 PM
gkistanova edited edge metadata.

Almost missed that. Sorry.

It seems you are taking a fully qualified path to a wrong directory.
Shouldn't it be for build_dir and not "."?

zorg/buildbot/builders/LLDBBuilder.py
94

Unless I'm missing something, it should be workdir=build_dir here.

This revision now requires changes to proceed.May 13 2016, 3:17 PM