[PATCH] D99057: [TableGen] Allow BitsInit to init integer in pseudo expansion
Paul C. Anagnostopoulos via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Mar 22 06:25:05 PDT 2021
Paul-C-Anagnostopoulos added inline comments.
================
Comment at: llvm/utils/TableGen/PseudoLoweringEmitter.cpp:111
++OpsAdded;
+ } else if (auto BI = dyn_cast<BitsInit>(Dag->getArg(i))) {
+ auto II = dyn_cast<IntInit>(BI->convertInitializerTo(IntRecTy::get()));
----------------
I would declare BI as BitsInit *BI, for consistency.
================
Comment at: llvm/utils/TableGen/PseudoLoweringEmitter.cpp:113
+ auto II = dyn_cast<IntInit>(BI->convertInitializerTo(IntRecTy::get()));
+ assert(II);
+ OperandMap[BaseIdx + i].Kind = OpData::Imm;
----------------
Please include a message in the assert, in the usual way.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D99057/new/
https://reviews.llvm.org/D99057
More information about the llvm-commits
mailing list