This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH H8300] Added -mno-exr option in case of function with "monitor" attribute
- From: Jeff Law <law at redhat dot com>
- To: Sandeep Kumar Singh <Sandeep dot Singh2 at kpitcummins dot com>
- Cc: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
- Date: Tue, 03 Apr 2012 10:55:01 -0600
- Subject: Re: [PATCH H8300] Added -mno-exr option in case of function with "monitor" attribute
- References: <00D82FEA4EAEB64782676AD908D06E6F1609A7C8@KCHJEXMB01.kpit.com>
On 04/03/2012 09:40 AM, Sandeep Kumar Singh wrote:
First, do you have an assignment on file with the FSF
We do have a corporate assignment for KPIT.
Excellent. Thanks for verifying.
My recollection was -mint32 was supported on the original H8/300;
is there something in particular that makes you want to issue an
error for that case?
My assumption was based on t-h8300 file. In this file, combination of
h8300 and int32 is not considered for MULTILIB_OPTIONS. Did I
misinterpret it? In that case I will remove the respective hunk from
the patch and will repost it.
Hmm, my memory must be faulty... You're absolutely correct, based on
the multilibs -mint32 isn't supported on the H8/300.
[law@stumpy gcc]$ ./xgcc -B./ -print-multi-lib
.;
h8300h;@mh
h8300s;@ms
h8sx;@msx
h8300h/normal;@mh@mn
h8300h/int32;@mh@mint32
h8300h/normal/int32;@mh@mn@mint32
h8300s/normal;@ms@mn
h8300s/int32;@ms@mint32
h8300s/normal/int32;@ms@mn@mint32
h8sx/normal;@msx@mn
h8sx/int32;@msx@mint32
h8sx/normal/int32;@msx@mn@mint32
In h8300.opt, rather than say "Push exr on stack", would it make
more sense to say "[Do not] Push extended registers on stack in
monitor functions"?
Thank you for this suggestion. I have made this change. Please find the
updated patch attached here.
I'll get it installed momentarily.
Thanks,
jeff