This is an archive of the discontinued LLVM Phabricator instance.

compiler-rt: use more __sanitizer_time_t on FreeBSD
ClosedPublic

Authored by emaste on Aug 26 2019, 10:37 AM.

Diff Detail

Repository
rL LLVM

Event Timeline

emaste created this revision.Aug 26 2019, 10:37 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 26 2019, 10:37 AM
Herald added a subscriber: dberris. · View Herald Transcript
This comment was removed by devnexen.
compiler-rt/lib/sanitizer_common/sanitizer_platform_limits_freebsd.h
117 ↗(On Diff #217195)

Would 'SANITIZER_WORDSIZE' be more appropriate ?

This comment was removed by devnexen.
devnexen accepted this revision.Aug 28 2019, 8:59 PM
This revision is now accepted and ready to land.Aug 28 2019, 8:59 PM

It turns out __LP64__ is wrong anyway, see D66758, so I intend to commit this first with no functional change and then follow up with D66758.

This revision was automatically updated to reflect the committed changes.