[llvm] r194485 - Remove unused variable.

Rafael EspĂ­ndola rafael.espindola at gmail.com
Tue Nov 12 08:51:59 PST 2013


On 12 November 2013 11:45, Sean Silva <silvas at purdue.edu> wrote:
> Did this trigger a compiler warning? This variable really shouldn't be
> unused since it's presence informs the YAMLIO templates that the output
> format should be "flow" style.
>
> In fact, removing that variable defeats the whole point of the
> `LLVM_YAML_IS_FLOW_SEQUENCE_VECTOR` macro vs.
> `LLVM_YAML_IS_SEQUENCE_VECTOR`.

Yes, current clang warn about. It only makes a difference on lld
tests, but unfortunately I didn't have it on the particular test
directory where I ran the tests.

It would be nice to have tests for this in llvm itself too :-(

Cheers,
Rafael



More information about the llvm-commits mailing list