This is an archive of the discontinued LLVM Phabricator instance.

[sanitizer_common] Rename SI_NOT_WINDOWS to SI_POSIX
ClosedPublic

Authored by mcgrathr on Jul 28 2017, 5:15 PM.

Details

Summary

New systems might be neither Windows nor POSIX. The SI_NOT_WINDOWS
macro in sanitizer_platform_interceptors.h was already effectively
the same as SI_POSIX, so just use SI_POSIX instead.

Diff Detail

Repository
rL LLVM

Event Timeline

mcgrathr created this revision.Jul 28 2017, 5:15 PM
vitalybuka accepted this revision.Jul 28 2017, 5:21 PM
This revision is now accepted and ready to land.Jul 28 2017, 5:21 PM
mcgrathr updated this revision to Diff 108751.Jul 28 2017, 5:21 PM

Missed two SI_NOT_WINDOWS cases.

I assume you don't have committer yet, so I need to land some changes?

This revision was automatically updated to reflect the committed changes.