On Aug 20, 2014, at 3:31 AM, Tony Wang <tony.w...@arm.com> wrote:
> It's a very simple test case modification to fix the test case failure on ARM 
> bare metal target.
> thread_local6.C is a test case to test the behavior of the deconstruct of a 
> thread local variable, and it just
> use _exit(0) to override the return 1(calling exit(1)). However, such a 
> behavior can't be detected on embedded
> target with dejagnu wrapper mechanism, because the wrapper will only output 
> the return value in exit() instead
> of _exit().

Hum, another solution might be to wrap _exit as well.  The patch is so simple 
and short, I’ll approve the posted patch; it is a nice step forward.  I’ll let 
you contemplate if you want to try a wrap on _exit.

Could you please add /* wrapping doesn’t reflect _exit value */ if you add this 
version.  This can go on line 3.  If _exit isn’t used, this is the clue we can 
then run the test case more often, and if someone wants to copy the use of 
_exit or fix wrapping, this is a handy clue for this.

Thanks.

Reply via email to