This is an archive of the discontinued LLVM Phabricator instance.

[lldb, tests] Correctly configure the yaml2obj paths
ClosedPublic

Authored by stella.stamenova on Sep 10 2020, 10:04 AM.

Details

Summary

They are currently not being set correctly for the case of multi-config generators like XCode and VS. There's also a typo in one of the cmake files.

Diff Detail

Event Timeline

Herald added a project: Restricted Project. · View Herald TranscriptSep 10 2020, 10:04 AM
stella.stamenova requested review of this revision.Sep 10 2020, 10:04 AM
JDevlieghere accepted this revision.Sep 10 2020, 10:06 AM

LGTM. This must've been broken for a while. I'll re-enable running the API tests on http://green.lab.llvm.org/green/view/LLDB/job/lldb-cmake-standalone/ so we can catch this earlier.

This revision is now accepted and ready to land.Sep 10 2020, 10:06 AM
This revision was landed with ongoing or failed builds.Sep 10 2020, 10:12 AM
This revision was automatically updated to reflect the committed changes.