[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