Investigate ASan/TSan errors #6396
Conversation
|
This does not seem tied to clang version: clang 14 and 15 error in the same manner. clang 16 does not install properly from the script, somehow the sanitizer library is not linkable, so I have not tested that one yet. |
This reverts commit bfe0345.
|
Clang 17 fixes asan! But not tsan... |
|
It looks like clang 18 fixes tsan as well (which I see makes sense based on this timeline). So, the best guess here is that something changed in the VMs and clang needs a very recent fix, from just a few months ago, to keep working. Bizarrely, I scanned many repos on github using sanitizers and none seem to hit this problem. I tried installing clang the way they did, I tried copying their flags, but nothing worked. Only using latest clang helps, but for all those other repos (top 10 results for github actions with asan) somehow things work without it... |
sbc100
left a comment
sbc100
left a comment
There was a problem hiding this comment.
I guess we still have plenty of coverage of building with older versions of clang in the other builders?
|
Yeah, we still use the default clang elsewhere which is 14. We also have gcc testing. |
2 participants
Since yesterday those jobs have been erroring out very early in an unclear way. asan just halts, and tsan says
My best guess is that something changed in the VM this runs on.