[PATCH] D95490: Support: Extract fs::resize_file_before_mapping_readwrite from FileOutputBuffer
Duncan P. N. Exon Smith via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jan 26 19:33:45 PST 2021
dexonsmith added a comment.
In D95490#2524082 <https://reviews.llvm.org/D95490#2524082>, @dexonsmith wrote:
> Note that I have no way of testing this myself on Windows; I'll be checking https://buildkite.com/llvm-project/premerge-checks/builds/23492.
This failed with:
C:\ws\w1\llvm-project\premerge-checks\llvm\unittests\Support\Path.cpp(1307): error: Expected: Status.getSize()
Which is: 4096
To be equal to: 123U
Which is: 123
which is the bug I fixed in the update https://reviews.llvm.org/differential/diff/319433/:
In D95490#2524095 <https://reviews.llvm.org/D95490#2524095>, @dexonsmith wrote:
> Fix a copy/paste bug in the test (only observable on Windows).
Anyone know how to trigger a rebuild now that the patch is fixed?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D95490/new/
https://reviews.llvm.org/D95490
More information about the llvm-commits
mailing list