[PATCH] D35686: [mips] Support -membedded-data and fix a related bug

Simon Atanasyan via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 20 14:49:25 PDT 2017


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

LGTM with minor nits.



================
Comment at: lib/Target/Mips/MipsTargetObjectFile.cpp:158
                                                        unsigned &Align) const {
-  if (IsConstantInSmallSection(DL, C, *TM))
+  if (IsConstantInSmallSection(DL, C, *TM))// && !EmbeddedData)
     return SmallDataSection;
----------------
Do we really need this comment? Its meaning is not obvious.


Repository:
  rL LLVM

https://reviews.llvm.org/D35686





More information about the llvm-commits mailing list