[PATCH] Missing break

Rafael Espíndola rafael.espindola at gmail.com
Sat Mar 29 20:33:55 PDT 2014


On 8 March 2014 16:57, Tobias Güntner
<guentner at rbg.informatik.tu-darmstadt.de> wrote:
> Hi!
>
> Am 08.03.2014 19:32, schrieb Tim Northover:
>
>> Both changes look obviously right, but the first one looks like it
>> should be testable somehow. Presumably you found it when the current
>> behaviour caused you problems. Do you have (or can you make) some
>> small code snippet that could be added to our tests?
>
>
> I'm afraid not. I was just browsing through the code, looking for something
> else, when I noticed a fall-through without a comment.

It does seem hard to test.

I have committed the patches as r205121 and r205119.

Cheers,
Rafael




More information about the llvm-commits mailing list