This is an archive of the discontinued LLVM Phabricator instance.

Fix an unused warning in WebAssemblyInstPrinter with NDEBUG.
ClosedPublic

Authored by sammccall on Oct 25 2016, 2:08 AM.

Reviewers

Commits

Summary

Fix an unused warning in WebAssemblyInstPrinter with NDEBUG.

Build Status

Buildable 758
Build 758: arc lint + arc unit

sammccall retitled this revision from to Fix an unused warning in WebAssemblyInstPrinter with NDEBUG..

sammccall updated this object.

sammccall added a reviewer: bkramer.

sammccall added a subscriber: llvm-commits.

lg, will commit for you.

This revision is now accepted and ready to land.Oct 25 2016, 2:17 AM

This revision was automatically updated to reflect the committed changes.

Path

Size

lib/

Target/

WebAssembly/

InstPrinter/

1 line

Show First 20 Lines • Show All 148 Lines • ▼ Show 20 Lines	if (Op.isReg()) {
if (OpNo < MII.get(MI->getOpcode()).getNumDefs())		if (OpNo < MII.get(MI->getOpcode()).getNumDefs())
O << '=';		O << '=';
} else if (Op.isImm()) {		} else if (Op.isImm()) {
const MCInstrDesc &Desc = MII.get(MI->getOpcode());		const MCInstrDesc &Desc = MII.get(MI->getOpcode());
assert((OpNo < Desc.getNumOperands() \|\|		assert((OpNo < Desc.getNumOperands() \|\|
(Desc.TSFlags & WebAssemblyII::VariableOpIsImmediate)) &&		(Desc.TSFlags & WebAssemblyII::VariableOpIsImmediate)) &&
"WebAssemblyII::VariableOpIsImmediate should be set for "		"WebAssemblyII::VariableOpIsImmediate should be set for "
"variable_ops immediate ops");		"variable_ops immediate ops");
		(void)Desc;
// TODO: (MII.get(MI->getOpcode()).TSFlags &		// TODO: (MII.get(MI->getOpcode()).TSFlags &
// WebAssemblyII::VariableOpImmediateIsLabel)		// WebAssemblyII::VariableOpImmediateIsLabel)
// can tell us whether this is an immediate referencing a label in the		// can tell us whether this is an immediate referencing a label in the
// control flow stack, and it may be nice to pretty-print.		// control flow stack, and it may be nice to pretty-print.
O << Op.getImm();		O << Op.getImm();
} else if (Op.isFPImm()) {		} else if (Op.isFPImm()) {
const MCInstrDesc &Desc = MII.get(MI->getOpcode());		const MCInstrDesc &Desc = MII.get(MI->getOpcode());
assert(OpNo < Desc.getNumOperands() &&		assert(OpNo < Desc.getNumOperands() &&
▲ Show 20 Lines • Show All 73 Lines • Show Last 20 Lines