[PATCH] D70559: [MIBundles] Move analyzePhysReg out of MIBundleOperands iterator (NFC).

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 2 08:05:59 PST 2019


arsenm accepted this revision.
arsenm added a comment.
This revision is now accepted and ready to land.

LGTM with nits



================
Comment at: llvm/include/llvm/CodeGen/MachineInstrBundle.h:261
+/// @returns A filled-in PhysRegInfo struct.
+PhysRegInfo AnalyzePhysRegInBundle(const MachineInstr &MI, unsigned Reg,
+                                   const TargetRegisterInfo *TRI);
----------------
s/Analyze/analyze.

Also should use Register instead of unsigned


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70559





More information about the llvm-commits mailing list