[PATCH] D84896: [MC] Add createMCInst() using a typed MCInst bump allocator.

Marcello Maggioni via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 29 18:30:21 PDT 2020


kariddi accepted this revision.
kariddi added inline comments.
This revision is now accepted and ready to land.


================
Comment at: llvm/lib/MC/MCContext.cpp:135
+MCInst *MCContext::createMCInst() {
+  return new(MCInstAllocator.Allocate()) MCInst;
+}
----------------
There is usually an extra space between the new and the "(" when a placement-like new is used in LLVM.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84896



More information about the llvm-commits mailing list