<div dir="ltr">David,<div><br></div><div>Would you do the honors? I don't have commit access.</div><div><br></div><div>It would be best, of course, if there was a way to phrase this so GCC doesn't die, but this pattern is everywhere so it might be futile to solve it only here.</div>
<div><br></div><div>- Kim</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Jul 31, 2014 at 9:03 PM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Probably worth adding a comment to exlpain why it needs to be there.<br>
<br>
Also, there's a reason that coding standard is in place - because we<br>
get better error (warning ) messages that way. Is there a way to<br>
workaround the bug without sacrificing future warning quality?<br>
<div class="HOEnZb"><div class="h5"><br>
On Thu, Jul 31, 2014 at 11:47 AM, Kim Gräsman <<a href="mailto:kim.grasman@gmail.com">kim.grasman@gmail.com</a>> wrote:<br>
> Hi Cy,<br>
><br>
> Unfortunately this goes against the general coding standards for LLVM:<br>
> <a href="http://llvm.org/docs/CodingStandards.html#don-t-use-default-labels-in-fully-covered-switches-over-enumerations" target="_blank">http://llvm.org/docs/CodingStandards.html#don-t-use-default-labels-in-fully-covered-switches-over-enumerations</a><br>

><br>
> but as a local workaround it seems fine.<br>
><br>
> I wonder if GCC 4.7 for ARM should be listed here:<br>
> <a href="http://llvm.org/docs/GettingStarted.html#check-here" target="_blank">http://llvm.org/docs/GettingStarted.html#check-here</a><br>
><br>
> - Kim<br>
><br>
><br>
> On Wed, Jul 30, 2014 at 10:57 AM, Cy Anetwork <<a href="mailto:cya3636@yahoo.com">cya3636@yahoo.com</a>> wrote:<br>
>><br>
>> Hello,<br>
>><br>
>> I've managed to track down some code which causes GCC to fail with an<br>
>> internal compiler error when compiling llvm/clang for ARM.  I've included<br>
>> the diff for this one file inline.. hope that's ok.<br>
>><br>
>> ==================================8<====================================<br>
>> --- ../llvm/lib/LTO/LTOCodeGenerator.cpp.old    2014-07-30 08:33:30 +0000<br>
>> +++ ../llvm/lib/LTO/LTOCodeGenerator.cpp        2014-07-30 08:39:03 +0000<br>
>> @@ -150,6 +150,9 @@<br>
>>    case LTO_DEBUG_MODEL_DWARF:<br>
>>      EmitDwarfDebugInfo = true;<br>
>>      return;<br>
>> +<br>
>> +  default:<br>
>> +    return;<br>
>>    }<br>
>>    llvm_unreachable("Unknown debug format!");<br>
>>  }<br>
>> @@ -161,6 +164,8 @@<br>
>>    case LTO_CODEGEN_PIC_MODEL_DYNAMIC_NO_PIC:<br>
>>      CodeModel = model;<br>
>>      return;<br>
>> +  default:<br>
>> +    return;<br>
>>    }<br>
>>    llvm_unreachable("Unknown PIC model!");<br>
>>  }<br>
>> @@ -293,6 +298,8 @@<br>
>>    case LTO_CODEGEN_PIC_MODEL_DYNAMIC_NO_PIC:<br>
>>      RelocModel = Reloc::DynamicNoPIC;<br>
>>      break;<br>
>> +  default:<br>
>> +    break;<br>
>>    }<br>
>><br>
>>    // construct LTOModule, hand over ownership of module and target<br>
>> ==================================8<====================================<br>
>><br>
>> Many thanks,<br>
>><br>
>> <a href="http://cya.net" target="_blank">cya.net</a><br>
>><br>
>><br>
>> _______________________________________________<br>
>> cfe-users mailing list<br>
>> <a href="mailto:cfe-users@cs.uiuc.edu">cfe-users@cs.uiuc.edu</a><br>
>> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-users" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-users</a><br>
><br>
><br>
><br>
> _______________________________________________<br>
> cfe-users mailing list<br>
> <a href="mailto:cfe-users@cs.uiuc.edu">cfe-users@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-users" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-users</a><br>
><br>
</div></div></blockquote></div><br></div>