[PATCH] D11072: Implement tool to convert bitcode to text.

[Karl Schimpf via llvm-commits]

kschimpf added a comment.

Kostya,

I agree that we should keep it simple. The text format isn't necessary, but
it does make writing unit tests much easier to maintain. Since the records
are simple lists of (textual) numbers, it is easy to update a record in
this form, should the format change.

When the code is in binary form, this is much harder to do.

However, for a first step, I agree that we should not worry about the
textual form. That can be left to be introduced later, when more concrete
examples of its utility can be shown.

I will also look at your suggested list of flow for mutators, to see how it
effects things.

I also agree that we want to keep things as simple as possible. We might be
able to put it all in a custom mutator library. I will look into that.

Finally, if possible, I will split this patch into smaller patches. I
initially presented as a single patch mainly to make it clear how the
pieces fit together.

Karl

Karl Schimpf


http://reviews.llvm.org/D11072