Update of bug #21972 (project freeciv):
Status: In Progress => Ready For Test
Planned Release: => 2.4.3, 2.5.0, 2.6.0
Follow-up Comment #2:
I still plan to do the more fundamental changes to the genhash code in TRUNK,
but attached patch already makes significant improvement only by replacing the
switch-case construct with carefully crafted if -constructs (or "carefully
crafted spaghetti", but given the benefits it's probably worth it). Minimize
the need to branch (check condition + jump instructions) code execution.
With this genhash_bucket_lookup() takes only 9% of the time (with the same
autogame) and the 3% saving goes almost directly to overall execution time of
As for profiling results in general, I'll post them to freeciv-dev later.
Additional Item Attachment:
File name: BucketIf.patch Size:2 KB
Reply to this item at:
Message sent via/by Gna!
Freeciv-dev mailing list