[PATCH] Driver: Replace switch cases with ifs.

Rui Ueyama ruiu at google.com
Fri Feb 6 18:52:47 PST 2015


The commit message of r188958 is very terse, it just says "add InputGraph
functionality", although it modified more than 1k lines of code. I'm
wondering why it was needed to convert ifs to cases at that moment. Do you
remember?

On Fri, Feb 6, 2015 at 6:46 PM, Shankar Easwaran <
shankar.kalpathi.easwaran at gmail.com> wrote:

> Ok. LGTM with those changes that you are done with.
>
>
> http://reviews.llvm.org/D7478
>
> EMAIL PREFERENCES
>   http://reviews.llvm.org/settings/panel/emailpreferences/
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150206/6163bbfa/attachment.html>


More information about the llvm-commits mailing list