[PATCH] D27579: [libFuzzer] Diff 19 - Automatically link with Windows library.

Zachary Turner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 9 04:00:09 PST 2016


zturner added a comment.

Can you do a quick sanity check that we return similar output on Posix and Windows (i.e. run all the above tests through Linux's `DirName` function and verify that they're the same)?  Obviously slashes will be different, and things like `\\?\` syntax and UNC syntax won't apply.  But for example, but the postconditions of the function should be the same on Posix and Windows, such as if you pass a simple filenmae like `foo.txt` to the function, then your example shows we return `.`, therefore Posix should return `.` as well otherwise someone might rely on it.  Similarly, for `foo\bar\file.txt`, if we return `foo\bar`, then I would expect `/foo/bar/file.txt` on Posix would return `/foo/bar` without a trailing slash.  (This is another argument for not relying on the Windows function in shlwapi I suppose, since we should try to match the postconditions as closely as possible.


Repository:
  rL LLVM

https://reviews.llvm.org/D27579





More information about the llvm-commits mailing list