public inbox for glibc-bugs-regex@sourceware.org
help / color / mirror / Atom feed
* [Bug regex/1279] New: regex simplification: "if (A == B) A += C - B;" -> "if (A == B) A = C;"
@ 2005-08-31 19:53 eggert at gnu dot org
2005-08-31 19:53 ` [Bug regex/1279] " eggert at gnu dot org
2005-09-06 23:37 ` drepper at redhat dot com
0 siblings, 2 replies; 3+ messages in thread
From: eggert at gnu dot org @ 2005-08-31 19:53 UTC (permalink / raw)
To: glibc-bugs-regex
While porting the regex code to 64-bit hosts I noticed an infelicity.
The code uses the equivalent of "if (A == B) A += C - B;";
it would be simpler and faster to use "if (A == B) A = C;".
This also happens to make the later changes simpler, but I thought I'd
separate it out since it makes sense regardless of the 64-bit changes.
I'll attach a patch.
--
Summary: regex simplification: "if (A == B) A += C - B;" -> "if
(A == B) A = C;"
Product: glibc
Version: 2.3.5
Status: NEW
Severity: normal
Priority: P2
Component: regex
AssignedTo: gotom at debian dot or dot jp
ReportedBy: eggert at gnu dot org
CC: glibc-bugs-regex at sources dot redhat dot com,glibc-
bugs at sources dot redhat dot com
http://sources.redhat.com/bugzilla/show_bug.cgi?id=1279
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.
^ permalink raw reply [flat|nested] 3+ messages in thread
* [Bug regex/1279] regex simplification: "if (A == B) A += C - B;" -> "if (A == B) A = C;"
2005-08-31 19:53 [Bug regex/1279] New: regex simplification: "if (A == B) A += C - B;" -> "if (A == B) A = C;" eggert at gnu dot org
@ 2005-08-31 19:53 ` eggert at gnu dot org
2005-09-06 23:37 ` drepper at redhat dot com
1 sibling, 0 replies; 3+ messages in thread
From: eggert at gnu dot org @ 2005-08-31 19:53 UTC (permalink / raw)
To: glibc-bugs-regex
------- Additional Comments From eggert at gnu dot org 2005-08-31 19:53 -------
Created an attachment (id=634)
--> (http://sources.redhat.com/bugzilla/attachment.cgi?id=634&action=view)
simplify update of rm_se, rm_eo
--
http://sources.redhat.com/bugzilla/show_bug.cgi?id=1279
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.
^ permalink raw reply [flat|nested] 3+ messages in thread
* [Bug regex/1279] regex simplification: "if (A == B) A += C - B;" -> "if (A == B) A = C;"
2005-08-31 19:53 [Bug regex/1279] New: regex simplification: "if (A == B) A += C - B;" -> "if (A == B) A = C;" eggert at gnu dot org
2005-08-31 19:53 ` [Bug regex/1279] " eggert at gnu dot org
@ 2005-09-06 23:37 ` drepper at redhat dot com
1 sibling, 0 replies; 3+ messages in thread
From: drepper at redhat dot com @ 2005-09-06 23:37 UTC (permalink / raw)
To: glibc-bugs-regex
------- Additional Comments From drepper at redhat dot com 2005-09-06 23:37 -------
The 64-bit portability part is bogus but the optimization is of course valid.
Although the compiler already performs it.
--
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution| |FIXED
http://sourceware.org/bugzilla/show_bug.cgi?id=1279
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2005-09-06 23:37 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2005-08-31 19:53 [Bug regex/1279] New: regex simplification: "if (A == B) A += C - B;" -> "if (A == B) A = C;" eggert at gnu dot org
2005-08-31 19:53 ` [Bug regex/1279] " eggert at gnu dot org
2005-09-06 23:37 ` drepper at redhat dot com
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).