This is an archive of the discontinued LLVM Phabricator instance.

Enable posix regex interceptors on linux.
ClosedPublic

Authored by eugenis on Dec 26 2018, 1:34 PM.

Diff Detail

Repository
rL LLVM

Event Timeline

eugenis created this revision.Dec 26 2018, 1:34 PM

While there, sanitizer_platform_limits_posix.h shall be divided with Darwin and renamed to sanitizer_platform_limits_linux.h. It's beyond this commit, but better to do it sooner than later.

Is the purpose of the proposed split to avoid all the #if-s for mac vs linux in that file? That would be nice, but we would also end up with a lot of code duplication, the same we have now between posix/netbsd/freebsd files. I'm not sure which is better.

This revision is now accepted and ready to land.Dec 27 2018, 1:43 PM

Is the purpose of the proposed split to avoid all the #if-s for mac vs linux in that file?

Yes.

That would be nice, but we would also end up with a lot of code duplication, the same we have now between posix/netbsd/freebsd files. I'm not sure which is better.

The other way around it's not maintainable, on the cost of some duplication.

This revision was automatically updated to reflect the committed changes.