[PATCH] D101125: [Debug-Info][NFC] add a wrapper for Die.addValue

[ChenZheng via Phabricator via llvm-commits]

shchenz added inline comments.


================
Comment at: llvm/lib/CodeGen/AsmPrinter/DwarfUnit.h:82
+                    dwarf::Attribute Attribute, dwarf::Form Form, T &&Value) {
+    Die.addValue(Alloc, DIEValue(Attribute, Form, std::forward<T>(Value)));
+  }
----------------
dblaikie wrote:
> Perhaps this function could use the DIEValueAllocator member directly, rather than having to pass it in - since all the callers pass the same value?
Good idea.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101125