diff options
author | Derek Schuff <dschuff@chromium.org> | 2013-02-04 14:56:34 -0800 |
---|---|---|
committer | Derek Schuff <dschuff@chromium.org> | 2013-02-04 14:56:34 -0800 |
commit | dbbbf3eb9d821f4ad3caab162ab70627fb32e10c (patch) | |
tree | 02efbe010c1a056a90271752379eb175715c5dce /lib/MC/MCAssembler.cpp | |
parent | 334e37be3e88c1d125bc833531a7d3056cc606a9 (diff) |
Remove the unnecessary hack and document the actually necessary one.
Diffstat (limited to 'lib/MC/MCAssembler.cpp')
-rw-r--r-- | lib/MC/MCAssembler.cpp | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/lib/MC/MCAssembler.cpp b/lib/MC/MCAssembler.cpp index 0b572eeca8..416af8b46f 100644 --- a/lib/MC/MCAssembler.cpp +++ b/lib/MC/MCAssembler.cpp @@ -241,7 +241,9 @@ MCSectionData::MCSectionData(const MCSection &_Section, MCAssembler *A) { // @LOCALMOD-BEGIN if (A) { - // TODO(dschuff): Carryover from previous localmods. still necessary? + // Necessary for IRT building because the IRT loader expects the end of + // the section to be bundle-aligned. Padding happens with 0's though, + // so it's not really ideal. TODO(dschuff) figure out how to do it right. A->getSectionList().push_back(this); if (A->isBundlingEnabled() && _Section.UseCodeAlign()) setAlignment(A->getBundleAlignSize()); |