replica
/
hanchenye-llvm-project
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

[PECOFF] Fix atom default alignment. 

Alignment(1) does not mean that the atom should be aligned on a 1 byte
boundary but on a 2^1 boundary. So, atoms without any specific alignment
requirements should have Alignment(0).

llvm-svn: 190723
This commit is contained in:
Rui Ueyama 2013-09-13 20:53:11 +00:00
parent 17e32460ed
commit 1ea9b97707
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
lld/lib/ReaderWriter/PECOFF/Atoms.h
View File

@ -100,7 +100,7 @@ public:
virtual StringRef name() const { return _name; }
virtual Interposable interposable() const { return interposeNo; }
virtual Merge merge() const { return mergeNo; }
virtual Alignment alignment() const { return Alignment(1); }
virtual Alignment alignment() const { return Alignment(0); }
virtual SectionChoice sectionChoice() const { return sectionBasedOnContent; }
virtual StringRef customSectionName() const { return ""; }
virtual SectionPosition sectionPosition() const { return sectionPositionAny; }
@ -152,7 +152,7 @@ public:
ContentPermissions perms, uint64_t ordinal)
: COFFBaseDefinedAtom(file, name, Kind::File), _sectionName(sectionName),
_scope(scope), _contentType(contentType), _permissions(perms),
_ordinal(ordinal), _alignment(1) {}
_ordinal(ordinal), _alignment(0) {}
static bool classof(const COFFBaseDefinedAtom *atom) {
return atom->getKind() == Kind::File;
@ -222,7 +222,7 @@ class COFFLinkerInternalAtom : public COFFBaseDefinedAtom {
public:
virtual uint64_t ordinal() const { return 0; }
virtual Scope scope() const { return scopeGlobal; }
virtual Alignment alignment() const { return Alignment(1); }
virtual Alignment alignment() const { return Alignment(0); }
virtual uint64_t size() const { return _data.size(); }
virtual ArrayRef<uint8_t> rawContent() const { return _data; }