[PATCH] D69536: Recommit "Add a heap alloc site marker field to the ExtraInfo in MachineInstrs"

Sid Manning via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 13 13:43:14 PST 2019


sidneym added a comment.

I get a build error with this patch (b288f7d6bb8fdd21d27ba755302db194c181fdaf <https://reviews.llvm.org/rGb288f7d6bb8fdd21d27ba755302db194c181fdaf>)  It looks like what got committed is a little different from above.  The committed patch partially changed updateCallSiteInfo with moveCallSiteInfo

  /local/mnt/workspace/tmp/tipcheck/llvm-project/llvm/lib/CodeGen/MachineFunction.cpp:825:23: error: 
        out-of-line definition of 'moveCallSiteInfo' does not match any declaration in
        'llvm::MachineFunction'; did you mean 'updateCallSiteInfo'?
  void MachineFunction::moveCallSiteInfo(const MachineInstr *Old,
                        ^~~~~~~~~~~~~~~~
                        updateCallSiteInfo


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69536





More information about the llvm-commits mailing list