[compiler-rt] r350139 - Add support for background thread on NetBSD in ASan

Kamil Rytarowski via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 7 09:33:59 PST 2019


On 07.01.2019 17:13, Kamil Rytarowski wrote:
> On 07.01.2019 16:38, Yvan Roux wrote:
>> Kamil, Diana,
>>
>> On Mon, 7 Jan 2019 at 15:43, Kamil Rytarowski via llvm-commits
>> <llvm-commits at lists.llvm.org> wrote:
>>>
>>> I've already replied to Yvan Roux 4 days ago:
>>
>> Sorry, I was preempted I didn't had time to investigate further.
>>
>>> It's not possible to guess what's wrong.
>>>  - Is the background thread started at all?
>>>  - Is __local_asan_dyninit optimized away or not executed?
>>>  - Is there a timing issue that test needs more time?
>>
>> Given that these test cases were OK in Thumbv8 build #1576 and the
>> commits involved into build #1577, my guess is that the issue is
>> related to __local_asan_dyninit.
>>
> 
> Can you debug it or give me remote access to a machine with this setup?
> 

If the answer is "no", then I will add a fallback for your setup to use
older branch.

>> build #1576 ninja check logs:
>> http://lab.llvm.org:8011/builders/clang-cmake-thumbv8-full-sh/builds/1576/steps/ninja%20check%202/logs/stdio
>>
>>> I don't have access to setup to reproduce it myself.
>>>
>>> The patch fixed Android setup and enabled NetBSD.
>>>
>>> I think that the problem is not caused by my change, but it's a side
>>> effect of other bug/optimization. I can mark the test as XFAIL for your
>>> setup or add ifdef for your Linux configuration to start the background
>>> thread in the same way as before.
>>>
>>>

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 850 bytes
Desc: OpenPGP digital signature
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190107/0fc57bd8/attachment.sig>


More information about the llvm-commits mailing list