[PATCH] D59663: [Support] MemoryBlock size should reflect the requested size

Andrew Ng via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 21 17:10:22 PDT 2019


andrewng added a comment.

> The stdout.s test you added in fact failed on Windows because this change was missing, and @thakis disabled it in rL351949 <https://reviews.llvm.org/rL351949>. So, I think we can just re-enable stdout.s to test this.

Yes, I was going to re-enable this test in a separate patch (as it's in LLD) and perhaps revert the changes to the WASM tests too. It was these WASM test failures that got me looking into this issue in the first place.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D59663/new/

https://reviews.llvm.org/D59663





More information about the llvm-commits mailing list