[perf-prof] Adjust source position addresses according to ELF header size

[perf-prof] Adjust source position addresses according to ELF header size

The "perf inject" command will place the generated function into a .text
section, placed directly after the ELF header. As a result, source position
addresses need to be adjusted according to the size of the ELF header, which is
0x40 for 64 bit architectures and 0x34 on 32 bit architectures.

We would previously adjust the addresses with 0x40 regardless of the
architecture.

BUG=

Review-Url: https://codereview.chromium.org/2783203005
Cr-Commit-Position: refs/heads/master@{#44325}
Committed: https://chromium.googlesource.com/v8/v8/+/3258b2690e1be50e2ab47722585ecfc29754a657

[Pierre Langlois, 2017-03-31 15:00:49]

Description was changed from

==========
[perf-prof] Adjust source position addresses according to ELF header size

The "perf inject" command will place the generated function into a .text
section, placed directly after the ELF header. As a result, source position
addresses need to be adjusted according to the size of the ELF header, which is
0x40 for 64 bit architectures and 0x34 on 32 bit architectures.

We would previously adjust the addresses with 0x40 regardless of the
architecture.

BUG=
==========

to

==========
[perf-prof] Adjust source position addresses according to ELF header size

The "perf inject" command will place the generated function into a .text
section, placed directly after the ELF header. As a result, source position
addresses need to be adjusted according to the size of the ELF header, which is
0x40 for 64 bit architectures and 0x34 on 32 bit architectures.

We would previously adjust the addresses with 0x40 regardless of the
architecture.

BUG=
==========

[Pierre Langlois, 2017-03-31 15:00:49]

pierre.langlois@arm.com changed reviewers:
+ danno@chromium.org, jarin@chromium.org

[Jarin, 2017-04-03 06:55:51]

lgtm. Thanks!

[Pierre Langlois, 2017-04-03 07:51:18]

The CQ bit was checked by pierre.langlois@arm.com

[commit-bot: I haz the power, 2017-04-03 07:51:27]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[danno, 2017-04-03 08:00:54]

lgtm. Thanks for spending the time to figure this all out!

[commit-bot: I haz the power, 2017-04-03 08:15:35]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1491205878841820, "parent_rev":
"a3be9e78c1bdb9332853cf37004697b737f1665f", "commit_rev":
"3258b2690e1be50e2ab47722585ecfc29754a657"}

[commit-bot: I haz the power, 2017-04-03 08:15:40]

Description was changed from

==========
[perf-prof] Adjust source position addresses according to ELF header size

The "perf inject" command will place the generated function into a .text
section, placed directly after the ELF header. As a result, source position
addresses need to be adjusted according to the size of the ELF header, which is
0x40 for 64 bit architectures and 0x34 on 32 bit architectures.

We would previously adjust the addresses with 0x40 regardless of the
architecture.

BUG=
==========

to

==========
[perf-prof] Adjust source position addresses according to ELF header size

The "perf inject" command will place the generated function into a .text
section, placed directly after the ELF header. As a result, source position
addresses need to be adjusted according to the size of the ELF header, which is
0x40 for 64 bit architectures and 0x34 on 32 bit architectures.

We would previously adjust the addresses with 0x40 regardless of the
architecture.

BUG=

Review-Url: https://codereview.chromium.org/2783203005
Cr-Commit-Position: refs/heads/master@{#44325}
Committed:
https://chromium.googlesource.com/v8/v8/+/3258b2690e1be50e2ab47722585ecfc2975...
==========

[commit-bot: I haz the power, 2017-04-03 08:15:41]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/v8/v8/+/3258b2690e1be50e2ab47722585ecfc2975...