Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(545)

Issue 341573003: Don't remove -m32 when building ia32 targets. (Closed)

Created:
6 years, 6 months ago by Nico
Modified:
6 years, 6 months ago
CC:
native-client-reviews_googlegroups.com
Visibility:
Public.

Description

Don't remove -m32 when building ia32 targets. There used to be a host_arch conditional that got lost when munge_phdr moved from chromium to nacl, see https://codereview.chromium.org/8800034/diff/4001/chrome/nacl.gypi https://codereview.chromium.org/8623002/diff/10001/src/trusted/service_runtime/linux/nacl_bootstrap.gyp Keeping the -m32 flag when target_arch and host_arch are both ia32 is important when building with a 32bit userspace on a 64bit kernel using a 64bit clang binary as compiler. As far as I can tell, -m32 and -sysroot are only added for target binaries, so this hack shouldn't be necesesary anymore. (And if it turns out that it still is for some reason, that reason can likely be fixed instead.) (Somewhat related to https://codereview.chromium.org/342493002/) BUG=368384 R=bradnelson@google.com Committed: https://src.chromium.org/viewvc/native_client?view=rev&revision=13369

Patch Set 1 #

Patch Set 2 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -16 lines) Patch
M src/trusted/service_runtime/linux/nacl_bootstrap.gyp View 1 1 chunk +0 lines, -16 lines 0 comments Download

Messages

Total messages: 23 (0 generated)
Nico
6 years, 6 months ago (2014-06-17 22:15:43 UTC) #1
Nico
+Brad, as it removes a TODO for him :-)
6 years, 6 months ago (2014-06-17 22:51:37 UTC) #2
bradn
+dyen David might recall differently, but I think this is fine. lgtm
6 years, 6 months ago (2014-06-17 23:21:36 UTC) #3
Mark Seaborn
Rubber-stamp LGTM Was that hack there because we used to support building Chromium with x86-32 ...
6 years, 6 months ago (2014-06-18 00:07:11 UTC) #4
Nico
On Tue, Jun 17, 2014 at 5:07 PM, <mseaborn@chromium.org> wrote: > Rubber-stamp LGTM > > ...
6 years, 6 months ago (2014-06-18 00:09:39 UTC) #5
Nico
(But if multilib still existed, I believe it should be possible to get that working ...
6 years, 6 months ago (2014-06-18 00:10:40 UTC) #6
Nico
Committed patchset #2 manually as r13369 (presubmit successful).
6 years, 6 months ago (2014-06-18 00:10:50 UTC) #7
Nico
This broke the tree: https://mail.google.com/mail/u/0/?ui=2#inbox/146ac52d19740582?compose=146ac5672d9f5837 Despite a similar sounding tryjob being green: http://build.chromium.org/p/tryserver.nacl/builders/nacl-precise32_glibc_opt/builds/3131 Does anyone ...
6 years, 6 months ago (2014-06-18 00:20:17 UTC) #8
Mark Seaborn
On 17 June 2014 17:20, Nico Weber <thakis@chromium.org> wrote: > This broke the tree: > ...
6 years, 6 months ago (2014-06-18 00:27:51 UTC) #9
Nico
On 2014/06/18 00:27:51, Mark Seaborn wrote: > On 17 June 2014 17:20, Nico Weber <mailto:thakis@chromium.org> ...
6 years, 6 months ago (2014-06-18 19:51:31 UTC) #10
Roland McGrath
On 2014/06/18 19:51:31, Nico (away) wrote: > So > http://build.chromium.org/p/client.nacl/builders/precise-32-glibc-dbg/builds/1335/steps/gyp_compile/logs/stdio > (which is before my ...
6 years, 6 months ago (2014-06-18 20:06:23 UTC) #11
Mark Seaborn
On 18 June 2014 12:51, <thakis@chromium.org> wrote: > On 2014/06/18 00:27:51, Mark Seaborn wrote: > ...
6 years, 6 months ago (2014-06-18 20:09:48 UTC) #12
native-client-reviews_googlegroups.com
On Wed, Jun 18, 2014 at 1:09 PM, Mark Seaborn <mseaborn@chromium.org> wrote: > x86-64 libelf ...
6 years, 6 months ago (2014-06-18 20:17:00 UTC) #13
Nico
Thanks for the replies! I had missed the GYP_DEFINES=target_arch=ia32 on the bot's output (despite trying ...
6 years, 6 months ago (2014-06-18 20:38:08 UTC) #14
khim
On Wed, Jun 18, 2014 at 4:07 AM, <mseaborn@chromium.org> wrote: > Rubber-stamp LGTM > > ...
6 years, 6 months ago (2014-06-18 20:50:46 UTC) #15
Nico
On Wed, Jun 18, 2014 at 1:50 PM, Victor Khimenko <khim@chromium.org> wrote: > > > ...
6 years, 6 months ago (2014-06-18 20:52:36 UTC) #16
khim
On Thu, Jun 19, 2014 at 12:52 AM, Nico Weber <thakis@chromium.org> wrote: > On Wed, ...
6 years, 6 months ago (2014-06-18 20:54:15 UTC) #17
native-client-reviews_googlegroups.com
The procedure is to add the requirement to chromium/src/build/install-build-deps.sh and then file an infrastructure ticket ...
6 years, 6 months ago (2014-06-18 20:55:13 UTC) #18
Nico
Huh, that's used by the nacl bots too? Ok, I'll give that a try then, ...
6 years, 6 months ago (2014-06-18 20:56:32 UTC) #19
Nico
I'm told that the script has been run on all linux slaves. I'll try relanding ...
6 years, 6 months ago (2014-06-24 22:26:41 UTC) #20
Nico
r13400. If the bots don't reject it immediately, I'll send a heads-up somewhere. (Where? nacl-eng?) ...
6 years, 6 months ago (2014-06-24 22:33:05 UTC) #21
Mark Seaborn
On 24 June 2014 15:33, Nico Weber <thakis@chromium.org> wrote: > r13400. If the bots don't ...
6 years, 6 months ago (2014-06-24 22:36:34 UTC) #22
Nico
6 years, 6 months ago (2014-06-24 22:44:15 UTC) #23
On Tue, Jun 24, 2014 at 3:36 PM, Mark Seaborn <mseaborn@chromium.org> wrote:

> On 24 June 2014 15:33, Nico Weber <thakis@chromium.org> wrote:
>
>> r13400. If the bots don't reject it immediately, I'll send a heads-up
>> somewhere. (Where? nacl-eng?)
>>
>
> native-client-dev is generally preferred over nacl-eng because it's the
> public mailing list:
> https://groups.google.com/forum/#!forum/native-client-dev
>

Thanks! "Your topic has been created and will appear after it has been
approved." :-)


>
> Cheers,
> Mark
>
>

-- 
You received this message because you are subscribed to the Google Groups
"Native-Client-Reviews" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to native-client-reviews+unsubscribe@googlegroups.com.
To post to this group, send email to native-client-reviews@googlegroups.com.
Visit this group at http://groups.google.com/group/native-client-reviews.
For more options, visit https://groups.google.com/d/optout.

Powered by Google App Engine
This is Rietveld 408576698