This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH] Use flag_general_regs_only with -mgeneral-regs-only

From: Uros Bizjak <ubizjak at gmail dot com>
To: Joseph Myers <joseph at codesourcery dot com>
Cc: "H.J. Lu" <hjl dot tools at gmail dot com>, Rainer Orth <ro at cebitec dot uni-bielefeld dot de>, "Koval, Julia" <julia dot koval at intel dot com>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>, "vaalfreja at gmail dot com" <vaalfreja at gmail dot com>, "law at redhat dot com" <law at redhat dot com>, "Zamyatin, Igor" <igor dot zamyatin at intel dot com>, Kirill Yukhin <kirill dot yukhin at gmail dot com>
Date: Wed, 25 May 2016 13:38:57 +0200
Subject: Re: [PATCH] Use flag_general_regs_only with -mgeneral-regs-only
Authentication-results: sourceware.org; auth=none
References: <CAMe9rOpjomeEMO+=8=XDyW+Q4SRVE1jPHL7xsvm5Ckqk3ySQcg at mail dot gmail dot com> <CAFULd4aksAjy1ejRu37U2abC5wxWSVUy4VsqzhNrtu7+gJ=VMw at mail dot gmail dot com> <CAMe9rOri+hx1E5RrPh30LNppNF77Y2r-C++cpGyKp7gk5ydTAQ at mail dot gmail dot com> <CAFULd4bnoiog=dUCcWG5M2h-rCfsYqdF55+0KSnPa-5X2c16rQ at mail dot gmail dot com> <CAMe9rOp5mWKj0J4gR=qczwXV0Z-6VSj6k7qBHGN9hnKn2w2yzA at mail dot gmail dot com> <CAFULd4aO1ugkKeX9NsadwW=6i6fUDB8G5=VfGK99+S7GyXxPzw at mail dot gmail dot com> <CAMe9rOrTqEKNZ23y13VHRk1GFMcLrZA+nEUOXpdLEwnUGP7Z8Q at mail dot gmail dot com> <CAFULd4brk56_if-tQsoJhkhgQv97Lw0ug22A46kakRW+nFWmPA at mail dot gmail dot com> <CAFULd4Z+-9Tx2nuY25EGByNQtQ8fy7047-Ub3S=-pydsUW=TKw at mail dot gmail dot com> <alpine dot DEB dot 2 dot 20 dot 1605242145310 dot 22670 at digraph dot polyomino dot org dot uk>

On Tue, May 24, 2016 at 11:47 PM, Joseph Myers <joseph@codesourcery.com> wrote:
> On Tue, 24 May 2016, Uros Bizjak wrote:
>
>> > I have thrown together a quick patch that defines target_flags as HOST_WIDE_INT.
>> >
>> > (Patch still needs a small correction, so opth-gen.awk will emit
>> > HOST_WIDE_INT_1 for MASK_* defines, have to go now, but I was able to
>> > compile functional x86_64-apple-darwin15.5.0 crosscompiler.)
>>
>> And here is attached complete (but untested!!) patch that should "just
>> work"(TM).
>
> Have you made sure that cl_host_wide_int gets set for options in
> target_flags, so that get_option_state, option_enabled etc. work correctly
> with such options?

Hm, no ...

Let's leave target_flags in its current state, and add additional
ix86_target_flags.

I'm currently testing attached patch, it also removes two unneded
option variables.

HJ, does this patch work for your?

Uros.

Attachment: p.diff.txt
Description: Text document

Follow-Ups:
- Re: [PATCH] Use flag_general_regs_only with -mgeneral-regs-only
  - From: H.J. Lu

References:
- [PATCH] Use flag_general_regs_only with -mgeneral-regs-only
  - From: H.J. Lu
- Re: [PATCH] Use flag_general_regs_only with -mgeneral-regs-only
  - From: Uros Bizjak
- Re: [PATCH] Use flag_general_regs_only with -mgeneral-regs-only
  - From: H.J. Lu
- Re: [PATCH] Use flag_general_regs_only with -mgeneral-regs-only
  - From: Uros Bizjak
- Re: [PATCH] Use flag_general_regs_only with -mgeneral-regs-only
  - From: H.J. Lu
- Re: [PATCH] Use flag_general_regs_only with -mgeneral-regs-only
  - From: Uros Bizjak
- Re: [PATCH] Use flag_general_regs_only with -mgeneral-regs-only
  - From: H.J. Lu
- Re: [PATCH] Use flag_general_regs_only with -mgeneral-regs-only
  - From: Uros Bizjak
- Re: [PATCH] Use flag_general_regs_only with -mgeneral-regs-only
  - From: Uros Bizjak
- Re: [PATCH] Use flag_general_regs_only with -mgeneral-regs-only
  - From: Joseph Myers

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]