Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
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.
I have nothing against the patch in general but I am uncomfortable with the fact that I can accidentally undo this change and very likely I will know nothing about breakages for weeks.
Build bot which run just few times a week could be super helpful.
Also I am personally not sure if such unpopular distribution is worth of supporting, But some patches are already accepted by llvm, so this one is LGTM.
I hope we will not need to add a lot of special cases in other sanitizers and just keep them as unsupported.