[PATCH] D54368: RegAllocFast: Record internal state based on register units

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 4 07:04:14 PDT 2020


arsenm marked an inline comment as done.
arsenm added inline comments.


================
Comment at: llvm/lib/CodeGen/RegAllocFast.cpp:136
     void setPhysRegState(MCPhysReg PhysReg, unsigned NewState);
+    bool isPhysRegFree(MCPhysReg PhysReg) const;
 
----------------
uabelho wrote:
> This seems to be unused? gcc warns with
> 
> ../lib/CodeGen/RegAllocFast.cpp:245:6: warning: 'bool {anonymous}::RegAllocFast::isPhysRegFree(llvm::MCPhysReg) const' defined but not used [-Wunused-function]
>  bool RegAllocFast::isPhysRegFree(MCPhysReg PhysReg) const {
>       ^~~~~~~~~~~~
> 
> Will it be used soon or can it be removed?
Removed


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

https://reviews.llvm.org/D54368





More information about the llvm-commits mailing list