[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