[PATCH] D95998: [Debug-Info] [NFC] move emit debug line unit length to MCStreamer class

[Igor Kudrin via Phabricator via llvm-commits]

ikudrin added a comment.

There is no need to create a new `MCStreamer::emitDwarfLineUnitLength()` method and make all the dependencies public. It is better to just use the already existed `MCStreamer::emitDwarfUnitLength()`. All you need to do is to create a temporary symbol and emit it right after calling `emitDwarfUnitLength()`.

By the way, the said `MCStreamer::emitDwarfUnitLength()` can be simplified so that it is called only with the `Hi` argument, without `Lo`. All the call sites create a temporary symbol, pass it to the method, and emitted it after that. All that can be done inside the method so that the calling code will be simplified. Your other patches in the set will also benefit from that change.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D95998