From: Dan Gohman Date: Mon, 3 Oct 2016 21:31:31 +0000 (+0000) Subject: [WebAssembly] Rename OPERAND_FP32IMM to OPERAND_F32IMM. X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=e0c2ec2a7e88c41622f5055b2b5faf0022c23294;p=llvm [WebAssembly] Rename OPERAND_FP32IMM to OPERAND_F32IMM. WebAssembly documentation consistently says "f32" rather than "fp32" to describe 32-bit floating-point. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@283146 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/lib/Target/WebAssembly/Disassembler/WebAssemblyDisassembler.cpp b/lib/Target/WebAssembly/Disassembler/WebAssemblyDisassembler.cpp index c0355aef0b3..4a9b93f7016 100644 --- a/lib/Target/WebAssembly/Disassembler/WebAssemblyDisassembler.cpp +++ b/lib/Target/WebAssembly/Disassembler/WebAssemblyDisassembler.cpp @@ -110,8 +110,8 @@ MCDisassembler::DecodeStatus WebAssemblyDisassembler::getInstruction( MI.addOperand(MCOperand::createReg(Reg)); break; } - case WebAssembly::OPERAND_FP32IMM: - case WebAssembly::OPERAND_FP64IMM: { + case WebAssembly::OPERAND_F32IMM: + case WebAssembly::OPERAND_F64IMM: { // TODO: MC converts all floating point immediate operands to double. // This is fine for numeric values, but may cause NaNs to change bits. if (Pos + sizeof(uint64_t) > Bytes.size()) diff --git a/lib/Target/WebAssembly/InstPrinter/WebAssemblyInstPrinter.cpp b/lib/Target/WebAssembly/InstPrinter/WebAssemblyInstPrinter.cpp index aadbf33123d..7fbca7e272c 100644 --- a/lib/Target/WebAssembly/InstPrinter/WebAssemblyInstPrinter.cpp +++ b/lib/Target/WebAssembly/InstPrinter/WebAssemblyInstPrinter.cpp @@ -171,12 +171,12 @@ void WebAssemblyInstPrinter::printOperand(const MCInst *MI, unsigned OpNo, assert(Desc.TSFlags == 0 && "WebAssembly variable_ops floating point ops don't use TSFlags"); const MCOperandInfo &Info = Desc.OpInfo[OpNo]; - if (Info.OperandType == WebAssembly::OPERAND_FP32IMM) { + if (Info.OperandType == WebAssembly::OPERAND_F32IMM) { // TODO: MC converts all floating point immediate operands to double. // This is fine for numeric values, but may cause NaNs to change bits. O << toString(APFloat(float(Op.getFPImm()))); } else { - assert(Info.OperandType == WebAssembly::OPERAND_FP64IMM); + assert(Info.OperandType == WebAssembly::OPERAND_F64IMM); O << toString(APFloat(Op.getFPImm())); } } else { diff --git a/lib/Target/WebAssembly/MCTargetDesc/WebAssemblyMCTargetDesc.h b/lib/Target/WebAssembly/MCTargetDesc/WebAssemblyMCTargetDesc.h index 7a32ddd5793..0f185638ff9 100644 --- a/lib/Target/WebAssembly/MCTargetDesc/WebAssemblyMCTargetDesc.h +++ b/lib/Target/WebAssembly/MCTargetDesc/WebAssemblyMCTargetDesc.h @@ -45,9 +45,9 @@ enum OperandType { /// Basic block label in a branch construct. OPERAND_BASIC_BLOCK = MCOI::OPERAND_FIRST_TARGET, /// 32-bit floating-point immediates. - OPERAND_FP32IMM, + OPERAND_F32IMM, /// 64-bit floating-point immediates. - OPERAND_FP64IMM, + OPERAND_F64IMM, /// p2align immediate for load and store address alignment. OPERAND_P2ALIGN }; diff --git a/lib/Target/WebAssembly/WebAssemblyInstrInfo.td b/lib/Target/WebAssembly/WebAssemblyInstrInfo.td index 922279d7742..0a6de2a52f5 100644 --- a/lib/Target/WebAssembly/WebAssemblyInstrInfo.td +++ b/lib/Target/WebAssembly/WebAssemblyInstrInfo.td @@ -71,10 +71,10 @@ let OperandNamespace = "WebAssembly" in { let OperandType = "OPERAND_BASIC_BLOCK" in def bb_op : Operand; -let OperandType = "OPERAND_FP32IMM" in +let OperandType = "OPERAND_F32IMM" in def f32imm_op : Operand; -let OperandType = "OPERAND_FP64IMM" in +let OperandType = "OPERAND_F64IMM" in def f64imm_op : Operand; let OperandType = "OPERAND_P2ALIGN" in {