S390: Change printf format specifier for size_t in heap.cc and spaces.cc

S390: Change printf format specifier for size_t in heap.cc and spaces.cc

On S390, GCC seems to treat size_t as 'long_unsigned_int'. Which conflicts with the %d format specifier since that expects an 'int'. So changing the macro to PRIuS. This seems to fix the issue while retaining the expected behaviour for all other systems.

R=mlippautz@chromium.org, hpayer@chromium.org
BUG=none

Review-Url: https://codereview.chromium.org/2759963003
Cr-Commit-Position: refs/heads/master@{#43988}
Committed: https://chromium.googlesource.com/v8/v8/+/e25ac28747d81f92d1f443213db2b87c3fd92f2c

[rayb, 2017-03-20 19:35:02]

ptal

[Michael Lippautz, 2017-03-21 08:10:25]

On 2017/03/20 19:35:02, rayb wrote:
> ptal

lgtm

[rayb, 2017-03-21 14:54:32]

The CQ bit was checked by rayb@ca.ibm.com

[commit-bot: I haz the power, 2017-03-21 14:54:35]

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

[commit-bot: I haz the power, 2017-03-21 16:07:01]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1490108072569080, "parent_rev":
"176a43fb86cc5b1f75751fc5b8011d0d77f84eb1", "commit_rev":
"e25ac28747d81f92d1f443213db2b87c3fd92f2c"}

[commit-bot: I haz the power, 2017-03-21 16:07:06]

Description was changed from

==========
S390: Change printf format specifier for size_t in heap.cc and spaces.cc

On S390, GCC seems to treat size_t as 'long_unsigned_int'. Which conflicts with
the %d format specifier since that expects an 'int'. So changing the macro to
PRIuS. This seems to fix the issue while retaining the expected behaviour for
all other systems.

R=mlippautz@chromium.org, hpayer@chromium.org
BUG=none
==========

to

==========
S390: Change printf format specifier for size_t in heap.cc and spaces.cc

On S390, GCC seems to treat size_t as 'long_unsigned_int'. Which conflicts with
the %d format specifier since that expects an 'int'. So changing the macro to
PRIuS. This seems to fix the issue while retaining the expected behaviour for
all other systems.

R=mlippautz@chromium.org, hpayer@chromium.org
BUG=none

Review-Url: https://codereview.chromium.org/2759963003
Cr-Commit-Position: refs/heads/master@{#43988}
Committed:
https://chromium.googlesource.com/v8/v8/+/e25ac28747d81f92d1f443213db2b87c3fd...
==========

[commit-bot: I haz the power, 2017-03-21 16:07:07]

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