From: DJ Delorie <dj@redhat.com>
To: Dave Brolley <brolley@redhat.com>,
Richard Sandiford <rdsandiford@googlemail.com>
Cc: cgen@sourceware.org
Subject: Re: intrinsics.scm patch
Date: Fri, 17 Apr 2009 02:07:00 -0000 [thread overview]
Message-ID: <200904170207.n3H27U4v031832@greed.delorie.com> (raw)
In-Reply-To: <49DE1424.6080000@redhat.com> (message from Dave Brolley on Thu, 09 Apr 2009 11:28:36 -0400)
> I can't comment on this, having never worked with it. Probably you and
> Richard Sandiford are the only ones who have worked on this.
As far as I know, only MeP uses it (or will, once the port is
accepted). Richard, do you have any comments on the patch? Or would
you like me to take over intrinsics.scm?
I did add another change, to include aliases in the intrinsics list.
2009-04-16 DJ Delorie <dj@redhat.com>
* intrinsics.scm (intrinsics-analyze!): Include aliases.
(guess-mode): Drop COP* syntax.
(need-insn): Add "--syscall--" to the list.
(enum-type): Drop COP* syntax.
(intrinsics.h): Likewise.
(intrinsic-protos.h): Change file header to allow it to be
included.
Index: intrinsics.scm
===================================================================
RCS file: /cvs/src/src/cgen/intrinsics.scm,v
retrieving revision 1.1
diff -p -U3 -r1.1 intrinsics.scm
--- intrinsics.scm 5 Nov 2007 15:46:19 -0000 1.1
+++ intrinsics.scm 17 Apr 2009 01:53:27 -0000
@@ -26,7 +26,7 @@
(define (intrinsics-analyze!)
(arch-analyze-insns! CURRENT-ARCH
- #f ; don't include aliases
+ #t ; include aliases
#t) ; do analyze the semantics
)
@@ -274,8 +274,8 @@
(cond
((equal? (md-operand:cdata op) 'FMAX_INT) "SI")
((equal? (md-operand:cdata op) 'FMAX_FLOAT) "SF")
- ((is-h-cr64? (md-operand:hw op)) "COPDI")
- ((is-h-cr? (md-operand:hw op)) "COPSI")
+ ((is-h-cr64? (md-operand:hw op)) "DI")
+ ((is-h-cr? (md-operand:hw op)) "SI")
((not (memory? (md-operand:type op))) "SI")
(else #f)))
@@ -1038,7 +1038,7 @@
;; Return true if the given insn should be included in the output files.
(define (need-insn? insn)
- (not (member (insn-mnemonic insn) '("--unused--" "--reserved--"))))
+ (not (member (insn-mnemonic insn) '("--unused--" "--reserved--" "--syscall--"))))
;; Set up global variables, if we haven't already.
(define (analyze-intrinsics!)
@@ -1268,8 +1268,8 @@
(define (enum-type op)
(cond
- ((is-h-cr64? (md-operand:hw op)) "cgen_regnum_operand_type_COPDI")
- ((is-h-cr? (md-operand:hw op)) "cgen_regnum_operand_type_COPSI")
+ ((is-h-cr64? (md-operand:hw op)) "cgen_regnum_operand_type_DI")
+ ((is-h-cr? (md-operand:hw op)) "cgen_regnum_operand_type_SI")
(else
(case (md-operand:cdata op)
((POINTER) "cgen_regnum_operand_type_POINTER")
@@ -1395,8 +1395,8 @@
" cgen_regnum_operand_type_USHORT, /* unsigned short */\n"
" cgen_regnum_operand_type_CHAR, /* char */\n"
" cgen_regnum_operand_type_UCHAR, /* unsigned char */\n"
- " cgen_regnum_operand_type_COPSI, /* __cop long */\n"
- " cgen_regnum_operand_type_COPDI, /* __cop long long */\n"
+ " cgen_regnum_operand_type_SI, /* __cop long */\n"
+ " cgen_regnum_operand_type_DI, /* __cop long long */\n"
" cgen_regnum_operand_type_CP_DATA_BUS_INT, /* cp_data_bus_int */\n"
" cgen_regnum_operand_type_DEFAULT = cgen_regnum_operand_type_LONG\n"
"};\n"
@@ -1530,20 +1530,14 @@
(else "long"))
(if (and (not (equal? (md-operand:cdata op) 'REGNUM))
(md-operand:write-index op))
- "&" "")))
+ "*" "")))
(define (intrinsic-protos.h) ; i.e., intrinsics.h
(string-write
"\n\n"
"/* DO NOT EDIT: This file is automatically generated by CGEN.\n"
" Any changes you make will be discarded when it is next regenerated.\n"
- "\n"
- " FURTHERMORE: this file serves only as documentation and must not be included\n "
- " in any application code which wishes to use intrinsics. */\n"
- "\n\n"
- "#error \"This file must not be included in any application code which uses intrinsics.\"\n"
- "#error \"It is here for documentation purposes ONLY.\"\n"
- "\n\n"
+ "*/\n\n"
"#if __MEP_CONFIG_CP_DATA_BUS_WIDTH == 64\n"
" typedef long long cp_data_bus_int;\n"
"#else\n"
next prev parent reply other threads:[~2009-04-17 2:07 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-04-08 21:09 DJ Delorie
2009-04-09 15:29 ` Dave Brolley
2009-04-17 2:07 ` DJ Delorie [this message]
2009-04-18 7:49 ` Richard Sandiford
2009-04-19 1:03 ` DJ Delorie
2009-04-20 15:13 ` Dave Brolley
2009-04-20 19:38 ` DJ Delorie
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=200904170207.n3H27U4v031832@greed.delorie.com \
--to=dj@redhat.com \
--cc=brolley@redhat.com \
--cc=cgen@sourceware.org \
--cc=rdsandiford@googlemail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).