On Friday, June 7, 2013, John McCall  wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Jun 2, 2013, at 1:11 AM, David Majnemer <<a href="javascript:;" onclick="_e(event, 'cvml', 'david.majnemer@gmail.com')">david.majnemer@gmail.com</a>> wrote:<br>

> Author: majnemer<br>
> Date: Sun Jun  2 03:11:22 2013<br>
> New Revision: 183084<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=183084&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=183084&view=rev</a><br>
> Log:<br>
> Properly consider the range of enum for range comparisons in C mode<br>
><br>
> In some cases, clang applies the C++ rules for computing the range of a<br>
> value when said value is an enum.<br>
><br>
> Instead, apply C semantics when in C mode.<br>
<br>
Was this proposed or reviewed somewhere?  This is totally wrong.</blockquote><div><br></div><div>It was proposed on this list in the form of a patch and LGTM'd. I then waited for more comments and received none.<span></span></div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
John.<br>
</blockquote>