This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Use flag_general_regs_only with -mgeneral-regs-only
- From: "H.J. Lu" <hjl dot tools at gmail dot com>
- To: Uros Bizjak <ubizjak at gmail dot com>
- Cc: 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: Tue, 24 May 2016 12:06:39 -0700
- 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>
On Tue, May 24, 2016 at 11:44 AM, Uros Bizjak <ubizjak@gmail.com> wrote:
> On Tue, May 24, 2016 at 8:15 PM, Uros Bizjak <ubizjak@gmail.com> wrote:
>> On Tue, May 24, 2016 at 7:18 PM, H.J. Lu <hjl.tools@gmail.com> wrote:
>>
>>>> Oh, target_flags is only a 32bit integer :(. Is there a reason it
>>>> can't be extended to HOST_WIDE_INT, as is the case with
>>>> ix86_isa_flags?
>>>
>>> target_flags is generic, not target specific. I want to limit my
>>> change to x86 backend and -mgeneral-regs-only doesn't need
>>> to use target_flags .
>>
>> 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).
>
-mgeneral-regs-only doesn't need to use target_flags and it shouldn't
use target_flags.
--
H.J.