[PATCH] D63320: [TableGen] Add "getOperandType" to get operand types from opcode/opidx

Nicolas Guillemot via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 18 15:38:18 PDT 2019


nlguillemot added a comment.

In D63320#1549309 <https://reviews.llvm.org/D63320#1549309>, @craig.topper wrote:

> Can the enum  be used without this? I'm trying to understand why the enum was there to begin with. Only one in tree target, AVR, defines GET_INSTRINFO_OPERAND_TYPES_ENUM. Its not built by default so I could't really check, but looking through the file that included it, I couldn't prove it was being used.


Indeed, I had the same question myself...

I couldn't find a particular way to use the enum otherwise myself, but I'm not an expert of the relevant code. `MCOperandInfo` does have a member that is somewhat misleadingly named `OperandType`, but that one seems to refer to `llvm::MCOI::OperandType` rather than this enum.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D63320/new/

https://reviews.llvm.org/D63320





More information about the llvm-commits mailing list