This is an archive of the discontinued LLVM Phabricator instance.

[LibFuzzer] Fix implementation of ``GetPeakRSSMb()`` on Mac OSX
ClosedPublic

Authored by delcypher on May 18 2016, 8:47 PM.

Details

Summary

[LibFuzzer]
Fix implementation of `GetPeakRSSMb()` on Mac OSX. On Linux
`rusage.ru_maxrss` is in KiB but on Mac OSX it is in
bytes.

Diff Detail

Repository
rL LLVM

Event Timeline

delcypher updated this revision to Diff 57731.May 18 2016, 8:47 PM
delcypher retitled this revision from to [LibFuzzer] Fix implementation of ``GetPeakRSSMb()`` on Mac OSX.
delcypher updated this object.
delcypher added reviewers: kcc, kubamracek.
delcypher added subscribers: llvm-commits, kcc, kubamracek.
kcc edited edge metadata.May 18 2016, 10:21 PM

same here, avoid C preprocessor at all costs, it is evil

delcypher updated this revision to Diff 57877.May 19 2016, 4:19 PM
delcypher edited edge metadata.

Remove use of #ifdefs

kcc accepted this revision.May 19 2016, 4:23 PM
kcc edited edge metadata.

LGTM

This revision is now accepted and ready to land.May 19 2016, 4:23 PM
This revision was automatically updated to reflect the committed changes.