[PATCH] D39770: NFC: Rename MCSafeSEHFragment to MCSymbolIdFragment

Reid Kleckner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 7 17:19:32 PST 2017


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

lgtm



================
Comment at: llvm/include/llvm/MC/MCFragment.h:565
 
-class MCSafeSEHFragment : public MCFragment {
+class MCSymbolIdFragment : public MCFragment {
   const MCSymbol *Sym;
----------------
Please add a doxygen comment here, something like:
  /// Represents a symbol table index fragment.

Really, I just want to see the words "symbol table index" somewhere, since they sound familiar and comforting. :)


https://reviews.llvm.org/D39770





More information about the llvm-commits mailing list