This is an archive of the discontinued LLVM Phabricator instance.

OpenBSD UBsan support final missing bits
ClosedPublic

Authored by devnexen on Mar 16 2018, 4:09 AM.

Details

Summary

One forgotten file change + reordering one header due to clang-format

Patch by David CARLIER

Diff Detail

Event Timeline

devnexen created this revision.Mar 16 2018, 4:09 AM

Sorry appearing obnoxious ... here the last needed changes hopefully. Thanks.

vitalybuka edited the summary of this revision. (Show Details)Mar 16 2018, 2:45 PM
vitalybuka added inline comments.Mar 16 2018, 2:59 PM
lib/sanitizer_common/sanitizer_procmaps_bsd.cc
22

Why do you need this order?

devnexen added inline comments.Mar 16 2018, 3:05 PM
lib/sanitizer_common/sanitizer_procmaps_bsd.cc
22

Because int32_t type needed for sysctl.

This revision was not accepted when it landed; it landed in state Needs Review.Mar 16 2018, 3:32 PM
This revision was automatically updated to reflect the committed changes.
vitalybuka reopened this revision.Mar 16 2018, 4:14 PM
vitalybuka accepted this revision.
This revision is now accepted and ready to land.Mar 16 2018, 4:14 PM
vitalybuka closed this revision.Mar 16 2018, 4:14 PM