[llvm-commits] [PATCH] Optimize memory usage of MC bundling

Jan Voung jvoung at chromium.org
Wed Jan 9 13:43:06 PST 2013



================
Comment at: include/llvm/MC/MCAssembler.h:249
@@ -214,1 +248,3 @@
 
+class MCCompactEncodedInstFragment : public MCEncodedFragment {
+  virtual void anchor();
----------------
Would it make sense to have a comment for this class?  Some of the other classes don't either, but some of them do.  The name of the class is already pretty descriptive...

================
Comment at: include/llvm/MC/MCAssembler.h:252
@@ +251,3 @@
+
+  bool AlignToBundleEnd;
+
----------------
Copy the comment for this field that you had in MCDataFragment?


http://llvm-reviews.chandlerc.com/D270



More information about the llvm-commits mailing list