[PATCH] D42334: [NFC] Refactor Apple Accelerator Tables

[Adrian Prantl via Phabricator via llvm-commits]

aprantl added inline comments.


================
Comment at: lib/CodeGen/AsmPrinter/DwarfAccelTable.h:324
+  DA.Name = Name;
+  DA.Values.push_back(new (Allocator) DataT(D));
+}
----------------
JDevlieghere wrote:
> aprantl wrote:
> > This is just me being curious: Would it make sense to have users pass the `new (Allocator) DataT(D)` and not make this a template at all?
> Wouldn't you agree that exposing the allocator to the caller makes a horrible interface? 
Fair point.


https://reviews.llvm.org/D42334