This is an archive of the discontinued LLVM Phabricator instance.

[ubsan][test] Don't disable ubsan testing on 64-bit Solaris/x86
ClosedPublic

Authored by ro on Jul 1 2019, 1:53 AM.

Details

Summary

Unlike asan, which isn't supported yet on 64-bit Solaris/x86, there's no reason to disable
ubsan. This patch does that, but keeps the 64-bit ubsan-with-asan tests disabled.

Tested on x86_64-pc-solaris2.11. Ok for trunk?

Diff Detail

Repository
rL LLVM

Event Timeline

ro created this revision.Jul 1 2019, 1:53 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 1 2019, 1:53 AM
Herald added subscribers: Restricted Project, fedor.sergeev, mgorny, kubamracek. · View Herald Transcript
ro added a comment.Jul 8 2019, 1:20 AM

Ping? It's been a week now and it should be pretty clear that 64-bit Solaris/x86 asan support is immaterial for ubsan.

This comment was removed by devnexen.
devnexen accepted this revision.Jul 8 2019, 1:47 AM
This revision is now accepted and ready to land.Jul 8 2019, 1:47 AM
This revision was automatically updated to reflect the committed changes.