[lldb-dev] problem with TestLoadUnload.py
Ted Woodward via lldb-dev
lldb-dev at lists.llvm.org
Fri Feb 23 13:08:50 PST 2018
I tried to put @skipIf(...) before setUp, but it didn't work. Currently I
have the build inside an if, checking for Hexagon. We don't support this use
of shared libraries, so all tests are skipped.
I certainly don't want to build the testcase 6 times, given that we're
moving away from that! But we need some general way to skip the build for
systems that can't build a given testcase. I have many tests skipped when
running without an OS, because we don't support c++11 in that mode.
Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a
Linux Foundation Collaborative Project
> -----Original Message-----
> From: aprantl at apple.com [mailto:aprantl at apple.com]
> Sent: Friday, February 23, 2018 2:47 PM
> To: Ted Woodward <ted.woodward at codeaurora.org>
> Cc: Pavel Labath <labath at google.com>
> Subject: Re: problem with TestLoadUnload.py
> > On Feb 23, 2018, at 12:08 PM, Ted Woodward
> <ted.woodward at codeaurora.org> wrote:
> > Hi Adrian,
> > In r324368 you changed TestLoadUnload.py to do a build in the setUp
> > function. This seems to be called before the actual testcases are
> > checked, so a skipped testcase would still call setUp and do the
> > build. If the build fails (say, on systems that don't have libgen.h),
> > the test errors out before it can be skipped.
> > Would you mind moving the build into each testcase?
> I'm afraid that would cause quite a performance degradation in the case
> the test is not skipped wince we then need to build the test once for each
> function. On the other hand since Pavel's change to run each test function
> parallel we are presumably building the test once per test function
> this should not be any more expensive.
> That said, I wonder if the right solution for your problem wouldn't be to
> setUp if the test depending on it is skipped.
> (Do you mind if I add lldb-dev to the CC list?)
> -- adrian
> > Thanks,
> > Ted
> > --
> > Qualcomm Innovation Center, Inc.
> > The Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
> > a Linux Foundation Collaborative Project
More information about the lldb-dev