[PATCH] D79570: [MC] Fix PR45805: infinite recursion in assembler

Eric Christopher via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 24 11:21:55 PDT 2020


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

One inline code comment otherwise lgtm.



================
Comment at: llvm/lib/MC/MCFragment.cpp:61
+    I = Sec->begin();
+  const MCFragment *FirstInvalidFragment = &*I;
+
----------------
A bit of vertical space here and bring the comment before the variable declaration here?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79570





More information about the llvm-commits mailing list