This is an archive of the discontinued LLVM Phabricator instance.

OpenBSD sanitizer common, define RLIMIT_AS constant
ClosedPublic

Authored by devnexen on Mar 3 2018, 4:20 AM.

Diff Detail

Repository
rL LLVM

Event Timeline

devnexen created this revision.Mar 3 2018, 4:20 AM
Herald added subscribers: Restricted Project, llvm-commits, kubamracek. · View Herald TranscriptMar 3 2018, 4:20 AM
krytarowski added inline comments.Mar 3 2018, 4:39 AM
lib/sanitizer_common/sanitizer_platform_limits_openbsd.h
34 ↗(On Diff #136911)

Do we need this check?

devnexen added inline comments.Mar 3 2018, 4:41 AM
lib/sanitizer_common/sanitizer_platform_limits_openbsd.h
34 ↗(On Diff #136911)

Not necessarily for now but might depend on future OpenBSD versions.

Knowing OpenBSD, their goal is no compatibility with others.

lib/sanitizer_common/sanitizer_platform_limits_openbsd.h
34 ↗(On Diff #136911)

I see no point in supporting hypothetical OpenBSD versions that are not there, especially since OpenBSD is known going its own route.

devnexen updated this revision to Diff 136920.Mar 3 2018, 7:04 AM
vitalybuka accepted this revision.Mar 3 2018, 5:24 PM
This revision is now accepted and ready to land.Mar 3 2018, 5:24 PM
This revision was automatically updated to reflect the committed changes.