[PATCH] D78088: [MIR] Add comments to INLINEASM immediate flag MachineOperands

Konstantin Schwarz via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 14 01:01:24 PDT 2020


kschwarz added a comment.

I've thought about adding enum to string conversion functions to InlineAsm.h, which could be reused from both MachineInstr.cpp and TargetInstrInfo.cpp to avoid code duplication.
Would that be preferred here?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78088





More information about the llvm-commits mailing list