[PATCH] D25046: AMDGPU: Emit runtime metadata version 2 as YAML

Yaxun Liu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 14 08:14:26 PST 2016


yaxunl added a comment.

>> Yes, I tried with llvm trunk. Did you run tests on windows or linux? For me tests are passing okay on linux but on windows they fail even with latest changes from trunk.

The failure on windows was due to output buffer not flushed in Program::Metadata::toYAML. The fix is to return Stream.str(). Thanks for catching this failure.


https://reviews.llvm.org/D25046





More information about the llvm-commits mailing list