[Lldb-commits] [PATCH] D77107: [intel-pt] Implement a basic test case
Greg Clayton via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Mon Mar 30 22:55:08 PDT 2020
clayborg added a comment.
I am worried if this test will be flaky on loaded machines. Not sure how we can ever guarantee we will see processor traces with our stuff in it if the machine is busy running many tests or even doing other things.
Comment at: lldb/test/API/tools/intel-features/intel-pt/test/TestIntelPTSimpleBinary.py:50-54
+ "rand", # We expect to see a reference to the rand function
+ # within the last instructions
+ hex(fun_start_adddress), # We expect to have seen the first
+ # instruction of 'fun'
+ "at main.cpp:21" # We expect to see the exit condition of
can we guarantee we will see any of these on a fully loaded machine running many tests simultaneously? Maybe we need to settle for the header of the output only to know that it tried to display something?
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
More information about the lldb-commits