[PATCH] D45822: [DEBUGINFO, NVPTX] Try to pack bytes data into a single string.

[Eric Christopher via Phabricator via llvm-commits]

echristo added inline comments.


================
Comment at: lib/MC/MCAsmStreamer.cpp:862-883
+    if (const char *DirectiveSeparator = MAI->getDirectiveSeparator()) {
+      unsigned NumElements = Data.size();
+      unsigned MaxLen = MAI->getMaxNumberOfElements()
+                            ? MAI->getMaxNumberOfElements()
+                            : NumElements;
+      unsigned NumChunks = 1 + (NumElements - 1) / MaxLen;
+      // Split the very long directives into several parts if the limit is
----------------
I'd prefer all of this (and basically the function) be sunk down into a target function for EmitBytes.


Repository:
  rL LLVM

https://reviews.llvm.org/D45822