[llvm] r183287 - Move BinaryRef to a new include/llvm/Object/YAML.h file.

Rafael EspĂ­ndola rafael.espindola at gmail.com
Thu Jun 6 05:35:09 PDT 2013


On 5 June 2013 18:12, Sean Silva <silvas at purdue.edu> wrote:
> How is it possible to use this class safely? isBinary is private state, so a
> caller can't know which method to call, and an assertion will fire if you
> call the wrong method.

The original idea was that obj2yaml knows it has a binary input and
yaml2obj knows it has a text input. It might be necessary to make it
three state (default constructed, binary, text).

Cheers,
Rafael



More information about the llvm-commits mailing list