[PATCH] D52127: [MF][MBB]: Add the ability to register callbacks for removal and insertion of MachineInstrs

Quentin Colombet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 17 09:12:13 PDT 2018

Previous message: [PATCH] D9400: llvm.noalias - Use noalias intrinsics when inlining (and update them when cloning metadata)
Next message: [PATCH] D52127: [MF][MBB]: Add the ability to register callbacks for removal and insertion of MachineInstrs
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

qcolombet added a comment.

Hi Aditya,

Could you use a delegate pattern like what we do for MachineRegisterInfio and LiveRangeEdit?

Other than that, the direction looks reasonable.

Cheers,
-Quentin


Repository:
  rL LLVM

https://reviews.llvm.org/D52127

Previous message: [PATCH] D9400: llvm.noalias - Use noalias intrinsics when inlining (and update them when cloning metadata)
Next message: [PATCH] D52127: [MF][MBB]: Add the ability to register callbacks for removal and insertion of MachineInstrs
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

More information about the llvm-commits mailing list