diff options
| author | Nicolas Geoffray <ngeoffray@google.com> | 2014-12-09 14:26:32 +0000 |
|---|---|---|
| committer | Nicolas Geoffray <ngeoffray@google.com> | 2014-12-09 14:49:02 +0000 |
| commit | 376b2bbf7c39108223a7a01568a7b4b04d84eeac (patch) | |
| tree | f6fdbe6180c3d330bc24aa35a7d9fa51f3eb1c55 /compiler/oat_writer.cc | |
| parent | b1a38e246cfbfb21100d9c9e57f33970e824f075 (diff) | |
| download | art-376b2bbf7c39108223a7a01568a7b4b04d84eeac.zip art-376b2bbf7c39108223a7a01568a7b4b04d84eeac.tar.gz art-376b2bbf7c39108223a7a01568a7b4b04d84eeac.tar.bz2 | |
Ensure stack maps are 4 byte aligned.
With the recent move to gcc 4.9, we are hitting alignment
SIGBUS on ARM. The reason is that gcc will optimize two consecutive
32bits loads into one 64bits load, and the instruction (ldrd)
will fail if the data is not aligned.
Also removed the emission of mapping table when a method is optimized.
The information can be found in the StackMap itself.
Change-Id: Icf79406c18a3f4db3c05d52fc2c0dd2e35bf0f8f
Diffstat (limited to 'compiler/oat_writer.cc')
| -rw-r--r-- | compiler/oat_writer.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/compiler/oat_writer.cc b/compiler/oat_writer.cc index 8a7abb4..6bb526c 100644 --- a/compiler/oat_writer.cc +++ b/compiler/oat_writer.cc @@ -530,7 +530,7 @@ struct OatWriter::GcMapDataAccess { struct OatWriter::MappingTableDataAccess { static const std::vector<uint8_t>* GetData(const CompiledMethod* compiled_method) ALWAYS_INLINE { - return &compiled_method->GetMappingTable(); + return compiled_method->GetMappingTable(); } static uint32_t GetOffset(OatClass* oat_class, size_t method_offsets_index) ALWAYS_INLINE { |