<p dir="ltr"><br>
On Jun 28, 2013 4:49 PM, "Richard Trieu" <<a href="mailto:rtrieu@google.com">rtrieu@google.com</a>> wrote:<br>
><br>
> Author: rtrieu<br>
> Date: Fri Jun 28 18:46:19 2013<br>
> New Revision: 185243<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=185243&view=rev">http://llvm.org/viewvc/llvm-project?rev=185243&view=rev</a><br>
> Log:<br>
> Change assert(0 && "text") to llvm_unreachable(0 && "text")<br>
><br>
> Modified:<br>
>     llvm/trunk/lib/Target/Hexagon/InstPrinter/HexagonInstPrinter.cpp<br>
><br>
> Modified: llvm/trunk/lib/Target/Hexagon/InstPrinter/HexagonInstPrinter.cpp<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Hexagon/InstPrinter/HexagonInstPrinter.cpp?rev=185243&r1=185242&r2=185243&view=diff">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Hexagon/InstPrinter/HexagonInstPrinter.cpp?rev=185243&r1=185242&r2=185243&view=diff</a><br>

> ==============================================================================<br>
> --- llvm/trunk/lib/Target/Hexagon/InstPrinter/HexagonInstPrinter.cpp (original)<br>
> +++ llvm/trunk/lib/Target/Hexagon/InstPrinter/HexagonInstPrinter.cpp Fri Jun 28 18:46:19 2013<br>
> @@ -179,7 +179,7 @@ void HexagonInstPrinter::printBranchOper<br>
>                                              raw_ostream &O) const {<br>
>    // Branches can take an immediate operand.  This is used by the branch<br>
>    // selection pass to print $+8, an eight byte displacement from the PC.<br>
> -  assert(0 && "Unknown branch operand.");<br>
> +  llvm_unreachable("Unknown branch operand.");<br>
>  }<br>
><br>
>  void HexagonInstPrinter::printCallOperand(const MCInst *MI, unsigned OpNo,<br>
> @@ -203,7 +203,7 @@ void HexagonInstPrinter::printSymbol(con<br>
>      O << '#';<br>
>      printOperand(MI, OpNo, O);<br>
>    } else {<br>
> -    assert(0 && "Unknown symbol operand");<br>
> +    llvm_unreachable("Unknown symbol operand");</p>
<p dir="ltr">This one should be able to be pushed up into an assert on whatever condition was attached to the 'if for which this is an 'else' (& drop the 'if/else' and the unreachable print here)</p>

<p dir="ltr">>      printOperand(MI, OpNo, O);<br>
>    }<br>
>    O << ')';<br>
><br>
><br>
> _______________________________________________<br>
> llvm-commits mailing list<br>
> <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</p>