<div dir="ltr"><div><div>Hi Bruno,<br><br></div>OK - I am on it.<br><br></div>-Nathan<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Nov 17, 2015 at 3:58 PM, Bruno Cardoso Lopes <span dir="ltr"><<a href="mailto:bruno.cardoso@gmail.com" target="_blank">bruno.cardoso@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Nathan,<br>
<br>
This is causing the warning:<br>
<br>
../tools/llvm-profdata/llvm-profdata.cpp:78:15: warning: 12<br>
enumeration values not handled in switch: 'success', 'eof',<br>
'unrecognized_format'... [-Wswitch]<br>
      switch (instrError) {<br>
              ^<br>
1 warning generated.<br>
<div class="HOEnZb"><div class="h5"><br>
On Tue, Nov 17, 2015 at 3:37 PM, Nathan Slingerland via llvm-commits<br>
<<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br>
> Author: slingn<br>
> Date: Tue Nov 17 17:37:09 2015<br>
> New Revision: 253400<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=253400&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=253400&view=rev</a><br>
> Log:<br>
> [llvm-profdata] Show hint for other mismatch errors when merging instr profdata<br>
><br>
> Missed bit of feedback from D14720.<br>
><br>
> Show the same "Make sure that all profile<br>
> data to be merged is generated from the same binary." hint for hash mismatch<br>
> and value site count mismatch as we now do for counter mismatch when merging<br>
> incompatible instrumentation profile data.<br>
><br>
> Modified:<br>
>     llvm/trunk/tools/llvm-profdata/llvm-profdata.cpp<br>
><br>
> Modified: llvm/trunk/tools/llvm-profdata/llvm-profdata.cpp<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/llvm-profdata/llvm-profdata.cpp?rev=253400&r1=253399&r2=253400&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/llvm-profdata/llvm-profdata.cpp?rev=253400&r1=253399&r2=253400&view=diff</a><br>
> ==============================================================================<br>
> --- llvm/trunk/tools/llvm-profdata/llvm-profdata.cpp (original)<br>
> +++ llvm/trunk/tools/llvm-profdata/llvm-profdata.cpp Tue Nov 17 17:37:09 2015<br>
> @@ -75,9 +75,13 @@ static void handleMergeWriterError(std::<br>
>      StringRef Hint = "";<br>
>      if (Error.category() == instrprof_category()) {<br>
>        instrprof_error instrError = static_cast<instrprof_error>(Error.value());<br>
> -      if (instrError == instrprof_error::count_mismatch) {<br>
> +      switch (instrError) {<br>
> +      case instrprof_error::hash_mismatch:<br>
> +      case instrprof_error::count_mismatch:<br>
> +      case instrprof_error::value_site_count_mismatch:<br>
>          Hint = "Make sure that all profile data to be merged is generated " \<br>
>                 "from the same binary.";<br>
> +        break;<br>
>        }<br>
>      }<br>
><br>
><br>
><br>
> _______________________________________________<br>
> llvm-commits mailing list<br>
> <a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
<br>
<br>
<br>
</div></div><span class="HOEnZb"><font color="#888888">--<br>
Bruno Cardoso Lopes<br>
<a href="http://www.brunocardoso.cc" rel="noreferrer" target="_blank">http://www.brunocardoso.cc</a><br>
</font></span></blockquote></div><br></div>