[PATCH] D19924: Representing bundle locked groups as fragments in MCAssembler

Derek Schuff via llvm-commits llvm-commits at lists.llvm.org
Wed May 11 11:27:29 PDT 2016


dschuff added a comment.

Petr, did you have some way of measuring memory use other than assembling a large module and using top or whatever to look at the RSS?


================
Comment at: include/llvm/MC/MCFragment.h:555
@@ +554,3 @@
+//   Prev  |##########|       F        |
+// -------------------------------------
+//                    ^
----------------
Is this illustration still true? Before, fragments with bundle padding had the "empty" padding space (the ###) followed by the content of the fragments. Now IIUC, there's just another fragment which is nothing but padding, correct? And now F->Offset just points to the beginning of the fragment, like any other fragment?


http://reviews.llvm.org/D19924





More information about the llvm-commits mailing list