Page MenuHomePhabricator

[Sanitizer] Undefined Behavior Sanitizer support proposal for DragonFlyBSD
Needs ReviewPublic

Authored by devnexen on Sat, Oct 17, 11:52 AM.

Details

Diff Detail

Event Timeline

devnexen created this revision.Sat, Oct 17, 11:52 AM
Herald added a project: Restricted Project. · View Herald TranscriptSat, Oct 17, 11:52 AM
Herald added subscribers: Restricted Project, phosek, jfb and 3 others. · View Herald Transcript
devnexen requested review of this revision.Sat, Oct 17, 11:52 AM
krytarowski requested changes to this revision.Sat, Oct 17, 1:19 PM

This contains unrelated code to ubsan/dragonflybsd. While there can you remove unsupported code for other OSs you introduced? That was never operational?

This revision now requires changes to proceed.Sat, Oct 17, 1:19 PM

This contains unrelated code to ubsan/dragonflybsd. While there can you remove unsupported code for other OSs you introduced? That was never operational?

Well DragonFly does not support sanitizers that s kind of mandatory or should I cut into different differential ?

Please, First remove unused + unusable code for already introduced OS. At most cutting it to ubsan.

devnexen updated this revision to Diff 298892.Sun, Oct 18, 11:31 AM

Focusing on ubsan change only