[PATCH] D84094: [DebugInfo] Fix misleading using of DWARF forms with DIELabel. NFCI.

Paul Robinson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 20 08:56:44 PDT 2020


probinson added a comment.

It looks like you have been tweaking a number of related SizeOf() methods.  Is it practical to have just one?  In a parent class, or even as a free function (as the size computation depends only on form/format, and is not related to the class at all).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84094





More information about the llvm-commits mailing list