On 8 June 2016 at 17:24, Omair Javaid <omair.jav...@linaro.org> wrote:
> On 26 May 2016 at 18:07, Pavel Labath <lab...@google.com> wrote:
>> These tests pass on soft float targets (e.g. android), so we should
>> not disable it there. I think we should figure out a way to make it
>> possible to disambiguate these. Right now it is possible to match
>> based on the triple of the target using a regular expression, but
>> maybe we could make that a bit easier. How about introducing a
>> "environment" variable, so that you could specify 'environment =
>> "eabihf"' or something like that?
>
> I have these tests failing on arm-linux-gnueabi (armel) and
> arm-linux-gnueabihf targets.
>
> I think putting in ABI as an environment variable is the right idea. I
> will see how I can separate out tests ABI based failures.
Sounds good.
>
>>
>>> ---
>>> lldb/trunk/packages/Python/lldbsuite/test/functionalities/archives/TestBSDArchives.py
>>> (original)
>>> +++
>>> lldb/trunk/packages/Python/lldbsuite/test/functionalities/archives/TestBSDArchives.py
>>> Wed May 25 13:48:39 2016
>>> @@ -21,6 +21,7 @@ class BSDArchivesTestCase(TestBase):
>>> self.line = line_number('a.c', '// Set file and line breakpoint
>>> inside a().')
>>>
>>> @expectedFailureAll(oslist=["windows"], bugnumber="llvm.org/pr24527.
>>> Makefile.rules doesn't know how to build static libs on Windows")
>>> + @expectedFailureAll(oslist=["linux"], archs=["arm"],
>>> bugnumber="llvm.org/pr27795")
>>
>> This test was passing in all configurations we are testing. The fact
>> that you are getting a unicode error here tells me that the problem is
>> probably specific to your setup (different locale or something). I
>> don't think we should be disabling tests on all arm builds because of
>> that. The problem is probably not that hard to fix, and it would be
>> extremely valuable to weed out system dependencies like this in order
>> to get more reproducible test results. Can you look into this ASAP?
>
> There exists a discussion regarding this unicode error.
> http://reviews.llvm.org/D16736
>
> I seem to be getting this on all kind of hardware i have right now.
All the more reason to not put in xfails for random configurations.
>
>>
>>> Modified:
>>> lldb/trunk/packages/Python/lldbsuite/test/linux/builtin_trap/TestBuiltinTrap.py
>>> URL:
>>> http://llvm.org/viewvc/llvm-project/lldb/trunk/packages/Python/lldbsuite/test/linux/builtin_trap/TestBuiltinTrap.py?rev=270745&r1=270744&r2=270745&view=diff
>>> ==============================================================================
>>> ---
>>> lldb/trunk/packages/Python/lldbsuite/test/linux/builtin_trap/TestBuiltinTrap.py
>>> (original)
>>> +++
>>> lldb/trunk/packages/Python/lldbsuite/test/linux/builtin_trap/TestBuiltinTrap.py
>>> Wed May 25 13:48:39 2016
>>> @@ -25,6 +25,7 @@ class BuiltinTrapTestCase(TestBase):
>>>
>>> @expectedFailureAll("llvm.org/pr15936", compiler="gcc",
>>> compiler_version=["<=","4.6"])
>>> @expectedFailureAll(archs="arm", compiler="gcc", triple=".*-android")
>>> # gcc generates incorrect linetable
>>> + @expectedFailureAll(oslist=['linux'], archs=['arm'])
>>> @skipIfWindows
>>> def test_with_run_command(self):
>>> """Test that LLDB handles a function with __builtin_trap
>>> correctly."""
>>>
>>
>> You're saying that the problem is due to gcc linetables. Then please
>> specify compiler="gcc" here (if you know an approximate version range,
>> even better).
>
> I will correct this.
Thanks.
pl
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
