Make gcmole correctly handle 32 bit architectures.

Make gcmole correctly handle 32 bit architectures.

NOTRY=true

Committed: https://crrev.com/7307bf0ff3b236308e5eee20b7a8bab26395d229
Cr-Commit-Position: refs/heads/master@{#26723}

[Michael Achenbach, 2015-02-18 09:25:48]

machenbach@chromium.org changed reviewers:
+ mstarzinger@chromium.org, svenpanne@chromium.org

[Michael Achenbach, 2015-02-18 09:25:49]

PTAL

[Michael Starzinger, 2015-02-18 09:41:55]

LGTM, if it works with the Clang built by tools/gcmole/bootstrap.sh as well.

https://codereview.chromium.org/935843003/diff/1/tools/gcmole/gcmole.lua
File tools/gcmole/gcmole.lua (right):

https://codereview.chromium.org/935843003/diff/1/tools/gcmole/gcmole.lua#newc...
tools/gcmole/gcmole.lua:121: cfg.clang_options)
nit: Maybe call this "arch_options" instead? WDYT?

[Michael Achenbach, 2015-02-18 11:14:13]

New patchsets have been uploaded after l-g-t-m from mstarzinger@chromium.org

[Michael Achenbach, 2015-02-18 11:15:29]

Done. Well, the boostrapper doesn't work on my workstation. Maybe it broke after
the trusty upgrade?

https://codereview.chromium.org/935843003/diff/1/tools/gcmole/gcmole.lua
File tools/gcmole/gcmole.lua (right):

https://codereview.chromium.org/935843003/diff/1/tools/gcmole/gcmole.lua#newc...
tools/gcmole/gcmole.lua:121: cfg.clang_options)
On 2015/02/18 09:41:54, Michael Starzinger wrote:
> nit: Maybe call this "arch_options" instead? WDYT?

Done.

[Michael Achenbach, 2015-02-18 14:49:40]

The CQ bit was checked by machenbach@chromium.org

[commit-bot: I haz the power, 2015-02-18 14:50:55]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/935843003/20001

[commit-bot: I haz the power, 2015-02-18 14:51:03]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-02-18 14:51:27]

Patchset 2 (id:??) landed as
https://crrev.com/7307bf0ff3b236308e5eee20b7a8bab26395d229
Cr-Commit-Position: refs/heads/master@{#26723}