[llvm] r194485 - Remove unused variable.

Sean Silva silvas at purdue.edu
Tue Nov 12 08:45:27 PST 2013


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`.

-- Sean Silva


On Tue, Nov 12, 2013 at 11:31 AM, Rafael Espindola <
rafael.espindola at gmail.com> wrote:

> Author: rafael
> Date: Tue Nov 12 10:31:59 2013
> New Revision: 194485
>
> URL: http://llvm.org/viewvc/llvm-project?rev=194485&view=rev
> Log:
> Remove unused variable.
>
> Modified:
>     llvm/trunk/include/llvm/Support/YAMLTraits.h
>
> Modified: llvm/trunk/include/llvm/Support/YAMLTraits.h
> URL:
> http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Support/YAMLTraits.h?rev=194485&r1=194484&r2=194485&view=diff
>
> ==============================================================================
> --- llvm/trunk/include/llvm/Support/YAMLTraits.h (original)
> +++ llvm/trunk/include/llvm/Support/YAMLTraits.h Tue Nov 12 10:31:59 2013
> @@ -1069,7 +1069,6 @@ operator<<(Output &yout, T &seq) {
>            seq.resize(index+1);
>    \
>          return seq[index];
>    \
>        }
>   \
> -      static const bool flow = true;
>    \
>      };
>    \
>    }
>   \
>    }
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20131112/1ad376c9/attachment.html>


More information about the llvm-commits mailing list