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

Amy Huang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 13 13:56:29 PST 2019


akhuang added a comment.

In D69536#1744703 <https://reviews.llvm.org/D69536#1744703>, @sidneym wrote:

> 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
>


Sorry, I guess I messed this up when merging.


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