* [CGEN PATCH] Rename TRACE_{EXTRACT,RESULT} to CGEN_TRACE_{EXTRACT,RESULT}
@ 2015-06-18 19:33 Doug Evans
0 siblings, 0 replies; 4+ messages in thread
From: Doug Evans @ 2015-06-18 19:33 UTC (permalink / raw)
To: cgen, gdb-patches
Hi.
This patch updates the generator so that the next time
the files are generated we don't lose the TRACE_{EXTRACT,RESULT}
renaming change.
ref:
https://sourceware.org/ml/gdb-patches/2015-06/msg00280.html
Tested with:
1) put guile 1.8.8 in $PATH
2) apply this patch:
https://sourceware.org/ml/gdb-patches/2015-06/msg00395.html
3)
bash$ $src/configure --prefix=$(pwd)/rel \
--enable-cgen-maint \
--enable-targets=all \
--enable-sim
bash$ make all-sim
I had to disable building of sim/m68hc11, and make a minor tweak to
sim/moxie,
so that their builds would complete.
2015-06-18 Doug Evans <dje@google.com>
* rtl-c.scm (rtl-c-set-trace): Update comment.
* sim-cpu.scm (cgen-semantics.c): Rename TRACE_RESULT to
CGEN_TRACE_RESULT.
(cgen-sem-switch.c): Ditto.
* sim-decode.scm (<hardware-base> 'gen-trace-extract): Update comment.
(<hw-register> 'gen-trace-extract): Ditto.
(<hw-address> 'gen-trace-extract): Ditto.
(/gen-op-trace-extract): Ditto.
(/gen-record-args): Rename TRACE_EXTRACT to CGEN_TRACE_EXTRACT.
* sim.scm (/op-gen-set-trace): Rename TRACE_RESULT to
CGEN_TRACE_RESULT.
(/op-gen-set-trace-parallel): Ditto.
(<operand> 'gen-set-trace): Ditto.
Index: rtl-c.scm
===================================================================
RCS file: /cvs/src/src/cgen/rtl-c.scm,v
retrieving revision 1.33
diff -u -p -u -p -r1.33 rtl-c.scm
--- rtl-c.scm 19 May 2014 01:24:00 -0000 1.33
+++ rtl-c.scm 18 Jun 2015 18:12:49 -0000
@@ -544,7 +544,7 @@
(rtl-c-get estate mode src))))
)
-; Same as rtl-c-set-quiet except also print TRACE_RESULT message.
+; Same as rtl-c-set-quiet except also print CGEN_TRACE_RESULT message.
; MODE is either a <mode> object or the mode name.
; ??? One possible change is to defer the (rtl-c-get src) call to dest's
; set handler. Such sources would be marked accordingly and rtl-c-get
Index: sim-cpu.scm
===================================================================
RCS file: /cvs/src/src/cgen/sim-cpu.scm,v
retrieving revision 1.19
diff -u -p -u -p -r1.19 sim-cpu.scm
--- sim-cpu.scm 23 Nov 2009 04:18:05 -0000 1.19
+++ sim-cpu.scm 18 Jun 2015 18:12:49 -0000
@@ -1090,8 +1090,8 @@ CGEN_ATTR_VALUE (NULL, abuf->idesc->attr
FAST_P, when desired, is defined on the command line, -DFAST_P=1. */
#if FAST_P
#define SEM_FN_NAME(cpu,fn) XCONCAT3 (cpu,_semf_,fn)
-#undef TRACE_RESULT
-#define TRACE_RESULT(cpu, abuf, name, type, val)
+#undef CGEN_TRACE_RESULT
+#define CGEN_TRACE_RESULT(cpu, abuf, name, type, val)
#else
#define SEM_FN_NAME(cpu,fn) XCONCAT3 (cpu,_sem_,fn)
#endif
@@ -1183,13 +1183,13 @@ CGEN_ATTR_VALUE (NULL, abuf->idesc->attr
/* If hyper-fast [well not unnecessarily slow] execution is selected, turn
off frills like tracing and profiling. */
-/* FIXME: A better way would be to have TRACE_RESULT check for something
+/* FIXME: A better way would be to have CGEN_TRACE_RESULT check for
something
that can cause it to be optimized out. Another way would be to emit
special handlers into the instruction \"stream\". */
#if FAST_P
-#undef TRACE_RESULT
-#define TRACE_RESULT(cpu, abuf, name, type, val)
+#undef CGEN_TRACE_RESULT
+#define CGEN_TRACE_RESULT(cpu, abuf, name, type, val)
#endif
#undef GET_ATTR
Index: sim-decode.scm
===================================================================
RCS file: /cvs/src/src/cgen/sim-decode.scm,v
retrieving revision 1.17
diff -u -p -u -p -r1.17 sim-decode.scm
--- sim-decode.scm 13 Feb 2010 03:39:15 -0000 1.17
+++ sim-decode.scm 18 Jun 2015 18:12:49 -0000
@@ -241,7 +241,7 @@ void
)
; gen-trace-extract method.
-; Return appropriate arguments for TRACE_EXTRACT.
+; Return appropriate arguments for CGEN_TRACE_EXTRACT.
(method-make!
<hardware-base> 'gen-trace-extract
@@ -266,7 +266,7 @@ void
""))
)
-; Return appropriate arguments for TRACE_EXTRACT.
+; Return appropriate arguments for CGEN_TRACE_EXTRACT.
(method-make!
<hw-register> 'gen-trace-extract
@@ -294,7 +294,7 @@ void
";\n"))
)
-; Return appropriate arguments for TRACE_EXTRACT.
+; Return appropriate arguments for CGEN_TRACE_EXTRACT.
(method-make!
<hw-address> 'gen-trace-extract
@@ -320,7 +320,7 @@ void
(send (op:type op) 'gen-extract op sfmt local?)
)
-; Return three of arguments to TRACE_EXTRACT:
+; Return three of arguments to CGEN_TRACE_EXTRACT:
; string argument to fprintf, character indicating type of third arg,
value.
; The type is one of: x.
@@ -374,7 +374,7 @@ void
iflds)
(string-list-map (lambda (op) (/gen-op-extract op sfmt #f))
operands)
- " TRACE_EXTRACT (current_cpu, abuf, (current_cpu, pc, "
+ " CGEN_TRACE_EXTRACT (current_cpu, abuf, (current_cpu, pc, "
"\"" (gen-sym sfmt) "\""
(string-list-map (lambda (f) (/gen-trace-argbuf-ifld f sfmt))
iflds)
Index: sim.scm
===================================================================
RCS file: /cvs/src/src/cgen/sim.scm,v
retrieving revision 1.32
diff -u -p -u -p -r1.32 sim.scm
--- sim.scm 25 Jan 2010 00:40:29 -0000 1.32
+++ sim.scm 18 Jun 2015 18:12:49 -0000
@@ -1260,12 +1260,12 @@
(send (op:type op) 'gen-set-quiet estate mode index selector
(cx:make-with-atlist mode "opval" (cx:atlist newval))))
(/op-gen-written-update op)
-; TRACE_RESULT_<MODE> (cpu, abuf, hwnum, opnum, value);
+; CGEN_TRACE_RESULT_<MODE> (cpu, abuf, hwnum, opnum, value);
; For each insn record array of operand numbers [or indices into
; operand instance table].
; Could just scan the operand table for the operand or hardware number,
; assuming the operand number is stored in `op'.
- " TRACE_RESULT (current_cpu, abuf"
+ " CGEN_TRACE_RESULT (current_cpu, abuf"
", " (send op 'gen-pretty-name mode)
", " (mode:printf-type mode)
", opval);\n"
@@ -1287,12 +1287,12 @@
" " /par-operand-macro " (" (gen-sym op) ")"
" = opval;\n"))
(/op-gen-written-update op)
-; TRACE_RESULT_<MODE> (cpu, abuf, hwnum, opnum, value);
+; CGEN_TRACE_RESULT_<MODE> (cpu, abuf, hwnum, opnum, value);
; For each insn record array of operand numbers [or indices into
; operand instance table].
; Could just scan the operand table for the operand or hardware number,
; assuming the operand number is stored in `op'.
- " TRACE_RESULT (current_cpu, abuf"
+ " CGEN_TRACE_RESULT (current_cpu, abuf"
", " (send op 'gen-pretty-name mode)
", " (mode:printf-type mode)
", opval);\n"
@@ -1323,7 +1323,8 @@
(/op-gen-set-quiet self estate mode index selector newval)))))
)
-; Return C code to set the value of an operand and print TRACE_RESULT
message.
+; Return C code to set the value of an operand and print CGEN_TRACE_RESULT
+; message.
; NEWVAL is a <c-expr> object of the value to store.
; If INDEX is non-#f use it, otherwise use (op:index self).
; This special handling of #f for INDEX is *only* supported for operands
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [CGEN PATCH] Rename TRACE_{EXTRACT,RESULT} to CGEN_TRACE_{EXTRACT,RESULT}
2015-06-18 19:33 Doug Evans
2015-06-19 14:37 ` Doug Evans
@ 2015-06-21 18:43 ` Mike Frysinger
1 sibling, 0 replies; 4+ messages in thread
From: Mike Frysinger @ 2015-06-21 18:43 UTC (permalink / raw)
To: Doug Evans; +Cc: cgen, gdb-patches
[-- Attachment #1: Type: text/plain, Size: 244 bytes --]
On 18 Jun 2015 19:33, Doug Evans wrote:
> This patch updates the generator so that the next time
> the files are generated we don't lose the TRACE_{EXTRACT,RESULT}
> renaming change.
patch looks fine to me, but i'm not a cgen maintainer
-mike
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [CGEN PATCH] Rename TRACE_{EXTRACT,RESULT} to CGEN_TRACE_{EXTRACT,RESULT}
2015-06-18 19:33 Doug Evans
@ 2015-06-19 14:37 ` Doug Evans
2015-06-21 18:43 ` Mike Frysinger
1 sibling, 0 replies; 4+ messages in thread
From: Doug Evans @ 2015-06-19 14:37 UTC (permalink / raw)
To: cgen, gdb-patches
On Thu, Jun 18, 2015 at 2:33 PM, Doug Evans <dje@google.com> wrote:
> [sorry for the repost, typo in email address]
>
> Hi.
>
> This patch updates the generator so that the next time
> the files are generated we don't lose the TRACE_{EXTRACT,RESULT}
> renaming change.
>
> ref:
> https://sourceware.org/ml/gdb-patches/2015-06/msg00280.html
>
> Tested with:
>
> 1) put guile 1.8.8 in $PATH
>
> 2) apply this patch:
>
> https://sourceware.org/ml/gdb-patches/2015-06/msg00395.html
>
> 3)
>
> bash$ $src/configure --prefix=$(pwd)/rel \
> --enable-cgen-maint \
> --enable-targets=all \
> --enable-sim
> bash$ make all-sim
Darn, I wanted these steps to be complete.
Missed a step, so let me try again.
1) put guile 1.8.8 in $PATH
[no special configure options are needed, just --prefix]
2) check out cgen
bash$ cd /place/to/put/cgen
bash$ cvs -d :pserver:anoncvs@sourceware.org:/cvs/src co cgen
2b) make sure the build can find it
bash$ ln -s $(pwd)/src/cgen $git/cgen
There's a TODO to be able to specify the path to cgen
when configuring binutils,gdb.
I've always just done this hack when I need it.
3) apply this patch
https://sourceware.org/ml/gdb-patches/2015-06/msg00395.html
4) Build all the sims.
Generated files that need regenerating will be,
including the opcodes files.
bash$ cd $build
bash$ $git/configure --prefix=$(pwd)/rel \
--enable-cgen-maint \
--enable-targets=all \
--enable-sim
bash$ make all-sim
If/when some form of (3) gets applied, and the TODO is done,
the steps should be simple enough.
Heads up, if you try this today there are two other outstanding patches
to cgen that affect the generated files.
^ permalink raw reply [flat|nested] 4+ messages in thread
* [CGEN PATCH] Rename TRACE_{EXTRACT,RESULT} to CGEN_TRACE_{EXTRACT,RESULT}
@ 2015-06-18 19:33 Doug Evans
2015-06-19 14:37 ` Doug Evans
2015-06-21 18:43 ` Mike Frysinger
0 siblings, 2 replies; 4+ messages in thread
From: Doug Evans @ 2015-06-18 19:33 UTC (permalink / raw)
To: cgen, gdb-patches
[sorry for the repost, typo in email address]
Hi.
This patch updates the generator so that the next time
the files are generated we don't lose the TRACE_{EXTRACT,RESULT}
renaming change.
ref:
https://sourceware.org/ml/gdb-patches/2015-06/msg00280.html
Tested with:
1) put guile 1.8.8 in $PATH
2) apply this patch:
https://sourceware.org/ml/gdb-patches/2015-06/msg00395.html
3)
bash$ $src/configure --prefix=$(pwd)/rel \
--enable-cgen-maint \
--enable-targets=all \
--enable-sim
bash$ make all-sim
I had to disable building of sim/m68hc11, and make a minor tweak to
sim/moxie,
so that their builds would complete.
2015-06-18 Doug Evans <dje@google.com>
* rtl-c.scm (rtl-c-set-trace): Update comment.
* sim-cpu.scm (cgen-semantics.c): Rename TRACE_RESULT to
CGEN_TRACE_RESULT.
(cgen-sem-switch.c): Ditto.
* sim-decode.scm (<hardware-base> 'gen-trace-extract): Update comment.
(<hw-register> 'gen-trace-extract): Ditto.
(<hw-address> 'gen-trace-extract): Ditto.
(/gen-op-trace-extract): Ditto.
(/gen-record-args): Rename TRACE_EXTRACT to CGEN_TRACE_EXTRACT.
* sim.scm (/op-gen-set-trace): Rename TRACE_RESULT to
CGEN_TRACE_RESULT.
(/op-gen-set-trace-parallel): Ditto.
(<operand> 'gen-set-trace): Ditto.
Index: rtl-c.scm
===================================================================
RCS file: /cvs/src/src/cgen/rtl-c.scm,v
retrieving revision 1.33
diff -u -p -u -p -r1.33 rtl-c.scm
--- rtl-c.scm 19 May 2014 01:24:00 -0000 1.33
+++ rtl-c.scm 18 Jun 2015 18:12:49 -0000
@@ -544,7 +544,7 @@
(rtl-c-get estate mode src))))
)
-; Same as rtl-c-set-quiet except also print TRACE_RESULT message.
+; Same as rtl-c-set-quiet except also print CGEN_TRACE_RESULT message.
; MODE is either a <mode> object or the mode name.
; ??? One possible change is to defer the (rtl-c-get src) call to dest's
; set handler. Such sources would be marked accordingly and rtl-c-get
Index: sim-cpu.scm
===================================================================
RCS file: /cvs/src/src/cgen/sim-cpu.scm,v
retrieving revision 1.19
diff -u -p -u -p -r1.19 sim-cpu.scm
--- sim-cpu.scm 23 Nov 2009 04:18:05 -0000 1.19
+++ sim-cpu.scm 18 Jun 2015 18:12:49 -0000
@@ -1090,8 +1090,8 @@ CGEN_ATTR_VALUE (NULL, abuf->idesc->attr
FAST_P, when desired, is defined on the command line, -DFAST_P=1. */
#if FAST_P
#define SEM_FN_NAME(cpu,fn) XCONCAT3 (cpu,_semf_,fn)
-#undef TRACE_RESULT
-#define TRACE_RESULT(cpu, abuf, name, type, val)
+#undef CGEN_TRACE_RESULT
+#define CGEN_TRACE_RESULT(cpu, abuf, name, type, val)
#else
#define SEM_FN_NAME(cpu,fn) XCONCAT3 (cpu,_sem_,fn)
#endif
@@ -1183,13 +1183,13 @@ CGEN_ATTR_VALUE (NULL, abuf->idesc->attr
/* If hyper-fast [well not unnecessarily slow] execution is selected, turn
off frills like tracing and profiling. */
-/* FIXME: A better way would be to have TRACE_RESULT check for something
+/* FIXME: A better way would be to have CGEN_TRACE_RESULT check for
something
that can cause it to be optimized out. Another way would be to emit
special handlers into the instruction \"stream\". */
#if FAST_P
-#undef TRACE_RESULT
-#define TRACE_RESULT(cpu, abuf, name, type, val)
+#undef CGEN_TRACE_RESULT
+#define CGEN_TRACE_RESULT(cpu, abuf, name, type, val)
#endif
#undef GET_ATTR
Index: sim-decode.scm
===================================================================
RCS file: /cvs/src/src/cgen/sim-decode.scm,v
retrieving revision 1.17
diff -u -p -u -p -r1.17 sim-decode.scm
--- sim-decode.scm 13 Feb 2010 03:39:15 -0000 1.17
+++ sim-decode.scm 18 Jun 2015 18:12:49 -0000
@@ -241,7 +241,7 @@ void
)
; gen-trace-extract method.
-; Return appropriate arguments for TRACE_EXTRACT.
+; Return appropriate arguments for CGEN_TRACE_EXTRACT.
(method-make!
<hardware-base> 'gen-trace-extract
@@ -266,7 +266,7 @@ void
""))
)
-; Return appropriate arguments for TRACE_EXTRACT.
+; Return appropriate arguments for CGEN_TRACE_EXTRACT.
(method-make!
<hw-register> 'gen-trace-extract
@@ -294,7 +294,7 @@ void
";\n"))
)
-; Return appropriate arguments for TRACE_EXTRACT.
+; Return appropriate arguments for CGEN_TRACE_EXTRACT.
(method-make!
<hw-address> 'gen-trace-extract
@@ -320,7 +320,7 @@ void
(send (op:type op) 'gen-extract op sfmt local?)
)
-; Return three of arguments to TRACE_EXTRACT:
+; Return three of arguments to CGEN_TRACE_EXTRACT:
; string argument to fprintf, character indicating type of third arg,
value.
; The type is one of: x.
@@ -374,7 +374,7 @@ void
iflds)
(string-list-map (lambda (op) (/gen-op-extract op sfmt #f))
operands)
- " TRACE_EXTRACT (current_cpu, abuf, (current_cpu, pc, "
+ " CGEN_TRACE_EXTRACT (current_cpu, abuf, (current_cpu, pc, "
"\"" (gen-sym sfmt) "\""
(string-list-map (lambda (f) (/gen-trace-argbuf-ifld f sfmt))
iflds)
Index: sim.scm
===================================================================
RCS file: /cvs/src/src/cgen/sim.scm,v
retrieving revision 1.32
diff -u -p -u -p -r1.32 sim.scm
--- sim.scm 25 Jan 2010 00:40:29 -0000 1.32
+++ sim.scm 18 Jun 2015 18:12:49 -0000
@@ -1260,12 +1260,12 @@
(send (op:type op) 'gen-set-quiet estate mode index selector
(cx:make-with-atlist mode "opval" (cx:atlist newval))))
(/op-gen-written-update op)
-; TRACE_RESULT_<MODE> (cpu, abuf, hwnum, opnum, value);
+; CGEN_TRACE_RESULT_<MODE> (cpu, abuf, hwnum, opnum, value);
; For each insn record array of operand numbers [or indices into
; operand instance table].
; Could just scan the operand table for the operand or hardware number,
; assuming the operand number is stored in `op'.
- " TRACE_RESULT (current_cpu, abuf"
+ " CGEN_TRACE_RESULT (current_cpu, abuf"
", " (send op 'gen-pretty-name mode)
", " (mode:printf-type mode)
", opval);\n"
@@ -1287,12 +1287,12 @@
" " /par-operand-macro " (" (gen-sym op) ")"
" = opval;\n"))
(/op-gen-written-update op)
-; TRACE_RESULT_<MODE> (cpu, abuf, hwnum, opnum, value);
+; CGEN_TRACE_RESULT_<MODE> (cpu, abuf, hwnum, opnum, value);
; For each insn record array of operand numbers [or indices into
; operand instance table].
; Could just scan the operand table for the operand or hardware number,
; assuming the operand number is stored in `op'.
- " TRACE_RESULT (current_cpu, abuf"
+ " CGEN_TRACE_RESULT (current_cpu, abuf"
", " (send op 'gen-pretty-name mode)
", " (mode:printf-type mode)
", opval);\n"
@@ -1323,7 +1323,8 @@
(/op-gen-set-quiet self estate mode index selector newval)))))
)
-; Return C code to set the value of an operand and print TRACE_RESULT
message.
+; Return C code to set the value of an operand and print CGEN_TRACE_RESULT
+; message.
; NEWVAL is a <c-expr> object of the value to store.
; If INDEX is non-#f use it, otherwise use (op:index self).
; This special handling of #f for INDEX is *only* supported for operands
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2015-06-21 18:43 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-06-18 19:33 [CGEN PATCH] Rename TRACE_{EXTRACT,RESULT} to CGEN_TRACE_{EXTRACT,RESULT} Doug Evans
2015-06-18 19:33 Doug Evans
2015-06-19 14:37 ` Doug Evans
2015-06-21 18:43 ` Mike Frysinger
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).