[PATCH] D109412: [MC] Make MCDwarfLineStr class public

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 9 17:55:53 PDT 2021


dblaikie added a comment.

Perhaps a unit test demonstrating the sort of thing you want to use this API for?

>  Note that MCDwarfLineTableHeader::Emit(), takes MCDwarfLineStr as an Optional<> parameter making it impossible to use the API if the class is not publicly defined.

Presumably you could still pass `None` to such an API? But yeah, couldn't use whatever functionality is exposed by passing a non-None value.


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

https://reviews.llvm.org/D109412



More information about the llvm-commits mailing list