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

Reid Kleckner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 21 15:57:26 PDT 2019


rnk added a subscriber: thakis.
rnk added a comment.

In D59663#1438487 <https://reviews.llvm.org/D59663#1438487>, @ruiu wrote:

> I'm wondering why it wasn't caught by a Windows bot if I missed it for Windows. Could you write a test that fails currently and passes after this change?


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.


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

https://reviews.llvm.org/D59663





More information about the llvm-commits mailing list