[Lldb-commits] [PATCH] D77452: [intel-pt] Improve the way the test determines whether to run
Pavel Labath via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Tue Apr 7 12:32:20 PDT 2020
labath added a reviewer: JDevlieghere.
labath added a subscriber: JDevlieghere.
labath added a comment.
Adding @JDevlieghere as he is in the same time zone and knows about all this stuff. This is on the right track, but changing lldb-dotest is not enough -- you'll also need to change `test/API/lit.cfg.py` as that's what's used for regular "check-lldb" runs. Also, instead of environment variables, please add a command line switch to dotest. Environment variables make it hard to reproduce steps when copy-pasting commands.
Comment at: lldb/utils/lldb-dotest/lldb-dotest.in:32
+ if lldb_build_intel_pt not in ["", "false", "off", "n", "no", "ignore"] \
+ and not lldb_build_intel_pt.endswith("-NOTFOUND"):
The usual solution to this is to run `llvm_canonicalize_cmake_booleans` in cmake.
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
More information about the lldb-commits