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