[PATCH] D85821: [MachOYAML] Simplify the section data emitting function. NFC.

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 12 02:20:35 PDT 2020


jhenderson accepted this revision.
jhenderson added a comment.
This revision is now accepted and ready to land.

Looks fine. I'm slightly nervous by the hard-coded `16` for the section name max length, but it's probably okay for now, and is no worse than what was there before.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85821



More information about the llvm-commits mailing list