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

[Amy Huang via Phabricator via llvm-commits]

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


================
Comment at: llvm/unittests/CodeGen/MachineInstrTest.cpp:144-145
+
+class BogusMCContext : public MCContext {
+public:
+  BogusMCContext()
----------------
rnk wrote:
> Why is the "bogus" derived class needed anyway? It seems like you could make a regular MCContext and it would be fine.
Good point. It is just a regular MCContext.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69536