public inbox for cgen@sourceware.org
 help / color / mirror / Atom feed
* CGEN:PATCH:RFA: gas testsuite generator
@ 2000-10-05 15:03 Dave Brolley
  2000-10-06  9:08 ` Doug Evans
  2000-10-06 10:10 ` Dave Brolley
  0 siblings, 2 replies; 3+ messages in thread
From: Dave Brolley @ 2000-10-05 15:03 UTC (permalink / raw)
  To: cgen

Hi,

The attached patch adds escapes for [ and ] characters in the
regular expressions which are used to compare the disassembly of
the cgen-generated allinsn testcase. It makes comparison for
insns like

	ld r1,@[sp+10]

proceed correctly.

OK to commit?

Dave
2000-10-05  Dave Brolley  <brolley@redhat.com>

	* gas-test.scm (gentest): Generate backslashes before '[' and ']'
	characters in the regular expression.

^ permalink raw reply	[flat|nested] 3+ messages in thread

* CGEN:PATCH:RFA: gas testsuite generator
  2000-10-05 15:03 CGEN:PATCH:RFA: gas testsuite generator Dave Brolley
@ 2000-10-06  9:08 ` Doug Evans
  2000-10-06 10:10 ` Dave Brolley
  1 sibling, 0 replies; 3+ messages in thread
From: Doug Evans @ 2000-10-06  9:08 UTC (permalink / raw)
  To: Dave Brolley; +Cc: cgen

Dave Brolley writes:
 > Hi,
 > 
 > The attached patch adds escapes for [ and ] characters in the
 > regular expressions which are used to compare the disassembly of
 > the cgen-generated allinsn testcase. It makes comparison for
 > insns like
 > 
 > 	ld r1,@[sp+10]
 > 
 > proceed correctly.
 > 
 > OK to commit?

ok.

 > Dave2000-10-05  Dave Brolley  <brolley@redhat.com>
 > 
 > 	* gas-test.scm (gentest): Generate backslashes before '[' and ']'
 > 	characters in the regular expression.
 > 
 > Index: cgen/gas-test.scm
 > ===================================================================
 > RCS file: /cvs/cvsfiles/devo/cgen/gas-test.scm,v
 > retrieving revision 1.25
 > diff -c -p -r1.25 gas-test.scm
 > *** gas-test.scm	2000/07/27 04:53:31	1.25
 > --- gas-test.scm	2000/10/05 20:29:39
 > *************** function gentest {
 > *** 190,196 ****
 >       echo \"#objdump: -dr\" >>${1}.d
 >       echo \"#name: $1\" >>${1}.d
 >       $BUILD/../binutils/objdump -dr a.out | \
 > ! 	sed -e 's/(/\\\\(/g' -e 's/)/\\\\)/g' -e 's/[+]/\\\\+/g' -e 's/[*]/\\\*/g' | \
 >   	sed -e 's/^.*file format.*$/.*: +file format .*/' \
 >   	>>${1}.d
 >       rm -f a.out
 > --- 190,201 ----
 >       echo \"#objdump: -dr\" >>${1}.d
 >       echo \"#name: $1\" >>${1}.d
 >       $BUILD/../binutils/objdump -dr a.out | \
 > ! 	sed -e 's/(/\\\\(/g' \
 > !             -e 's/)/\\\\)/g' \
 > !             -e 's/\\[/\\\\\\[/g' \
 > !             -e 's/\\]/\\\\\\]/g' \
 > !             -e 's/[+]/\\\\+/g' \
 > !             -e 's/[*]/\\\\*/g' | \
 >   	sed -e 's/^.*file format.*$/.*: +file format .*/' \
 >   	>>${1}.d
 >       rm -f a.out

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: CGEN:PATCH:RFA: gas testsuite generator
  2000-10-05 15:03 CGEN:PATCH:RFA: gas testsuite generator Dave Brolley
  2000-10-06  9:08 ` Doug Evans
@ 2000-10-06 10:10 ` Dave Brolley
  1 sibling, 0 replies; 3+ messages in thread
From: Dave Brolley @ 2000-10-06 10:10 UTC (permalink / raw)
  To: cgen

Approved by Frank Eigler and Doug Evans and committed.

Dave

Dave Brolley wrote:
> 
> Hi,
> 
> The attached patch adds escapes for [ and ] characters in the
> regular expressions which are used to compare the disassembly of
> the cgen-generated allinsn testcase. It makes comparison for
> insns like
> 
>         ld r1,@[sp+10]
> 
> proceed correctly.
> 
> OK to commit?
> 
> Dave
> 
>   -----------------------------------------------------------------
> 2000-10-05  Dave Brolley  <brolley@redhat.com>
> 
>         * gas-test.scm (gentest): Generate backslashes before '[' and ']'
>         characters in the regular expression.
> 
>   -----------------------------------------------------------------
> Index: cgen/gas-test.scm
> ===================================================================
> RCS file: /cvs/cvsfiles/devo/cgen/gas-test.scm,v
> retrieving revision 1.25
> diff -c -p -r1.25 gas-test.scm
> *** gas-test.scm        2000/07/27 04:53:31     1.25
> --- gas-test.scm        2000/10/05 20:29:39
> *************** function gentest {
> *** 190,196 ****
>       echo \"#objdump: -dr\" >>${1}.d
>       echo \"#name: $1\" >>${1}.d
>       $BUILD/../binutils/objdump -dr a.out | \
> !       sed -e 's/(/\\\\(/g' -e 's/)/\\\\)/g' -e 's/[+]/\\\\+/g' -e 's/[*]/\\\*/g' | \
>         sed -e 's/^.*file format.*$/.*: +file format .*/' \
>         >>${1}.d
>       rm -f a.out
> --- 190,201 ----
>       echo \"#objdump: -dr\" >>${1}.d
>       echo \"#name: $1\" >>${1}.d
>       $BUILD/../binutils/objdump -dr a.out | \
> !       sed -e 's/(/\\\\(/g' \
> !             -e 's/)/\\\\)/g' \
> !             -e 's/\\[/\\\\\\[/g' \
> !             -e 's/\\]/\\\\\\]/g' \
> !             -e 's/[+]/\\\\+/g' \
> !             -e 's/[*]/\\\\*/g' | \
>         sed -e 's/^.*file format.*$/.*: +file format .*/' \
>         >>${1}.d
>       rm -f a.out

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2000-10-06 10:10 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2000-10-05 15:03 CGEN:PATCH:RFA: gas testsuite generator Dave Brolley
2000-10-06  9:08 ` Doug Evans
2000-10-06 10:10 ` Dave Brolley

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).