[PATCH] D87010: [DebugInfo] Add new emitting methods for values which depend on the DWARF format (3/19).
    Igor Kudrin via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Wed Sep  2 22:59:26 PDT 2020
    
    
  
ikudrin added a comment.
In D87010#2252619 <https://reviews.llvm.org/D87010#2252619>, @dblaikie wrote:
> Might be too fine-grained, may be better off rolling this into whatever patch uses this functionality & testing it end-to-end there, rather than with a unit test? (unless there are cases that are especially hard to reach via the usual functional tests)
That is possible, but that would tie patches for independent tables closer than necessary. That would also make patches that introduce the functions less focused.
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D87010/new/
https://reviews.llvm.org/D87010
    
    
More information about the llvm-commits
mailing list