This is an archive of the discontinued LLVM Phabricator instance.

[sanitizer_common] Move filesystem-related code out of sanitizer_common.cc
ClosedPublic

Authored by mcgrathr on Jul 18 2017, 4:49 PM.

Diff Detail

Repository
rL LLVM

Event Timeline

mcgrathr created this revision.Jul 18 2017, 4:49 PM
alekseyshl accepted this revision.Jul 19 2017, 12:06 PM
alekseyshl added inline comments.
lib/sanitizer_common/CMakeLists.txt
73 ↗(On Diff #107207)

Add sanitizer_file.h to this set

lib/sanitizer_common/sanitizer_file.h
10 ↗(On Diff #107207)

Please add here the rationale for the existence of this file (from the patch description).

This revision is now accepted and ready to land.Jul 19 2017, 12:06 PM
mcgrathr updated this revision to Diff 107353.Jul 19 2017, 12:19 PM

Added more comments at top of new files.
Added sanitizer_file.h to CMakeLists.txt.

Can you land it for me?

Thanks,
Roland

This revision was automatically updated to reflect the committed changes.
This revision is now accepted and ready to land.Jul 20 2017, 11:26 AM
mcgrathr updated this revision to Diff 107609.Jul 20 2017, 5:02 PM

Added missing #include's for Windows and Mac.

I've added the #include's that I think from inspection might fix the Windows and Mac builds, but I'm not in a position to test those builds easily.
If someone can test those builds so this can re-land, that would be great.

I have no access to Win or Mac either but I'll ask around. Or we can just land it and revert if something goes wrong again...

phosek is setting up a Windows box and said he'd test it for me when he gets that in shape. (I think he has a Mac too. :-)
But I'm also fairly confident that my fix will have covered the build breakage, so landing again optimistically will most likely be just fine.

Thanks,
Roland

This revision was automatically updated to reflect the committed changes.