[PATCH] D30156: llvm-mc-fuzzer: add support for assembly

[Daniel Sanders via Phabricator via llvm-commits]

dsanders added a comment.

In https://reviews.llvm.org/D30156#681790, @bcain wrote:

> In https://reviews.llvm.org/D30156#681438, @dsanders wrote:
>
> > > Currently we just attempt assembly and ignore the result.
> >
> > Ignoring the result is the right thing to do since failure to assemble is an expected response to some inputs. Whether it's a correct response to a particular input can be found by separately running the corpus through the assembler and comparing against a reference (most likely, another assembler).
>
>
> Just to make sure we're all on the same page, I think anything leveraging a reference assembler is out of scope (for now anyways).


That's right. Crashes aside, it's not llvm-mc-fuzzers job to check the correctness of any particular output. The comment about a reference assembler was meant to indicate how someone would check correctness externally using the corpus emitted by llvm-mc-fuzzer.


Repository:
  rL LLVM

https://reviews.llvm.org/D30156