Opened 11 years ago

Closed 11 years ago

#2024 closed defect (wontfix)

[PATCH] branching in minimap texture filling

Reported by: tuan kuranes Owned by: tuan kuranes
Priority: Nice to Have Milestone: Alpha 15
Component: Core engine Keywords: patch
Cc: Patch:

Description (last modified by tuan kuranes)

Avoiding "if" inside big loops allows CPU to avoid branch mispredictions. In this case, just memsetting all to 0 prior the loop avoid some if in the loop code just to fill it with zeroes.

Attachments (1)

branch.patch (3.2 KB ) - added by tuan kuranes 11 years ago.

Download all attachments as: .zip

Change History (7)

by tuan kuranes, 11 years ago

Attachment: branch.patch added

comment:1 by tuan kuranes, 11 years ago

Description: modified (diff)
Summary: branching in minimap texture filling[PATCH] branching in minimap texture filling

comment:2 by Kieran P, 11 years ago

Keywords: patch review added
Milestone: BacklogAlpha 14
Priority: Should HaveNice to Have

comment:3 by historic_bruno, 11 years ago

Owner: set to tuan kuranes

comment:4 by Kieran P, 11 years ago

Milestone: Alpha 14Alpha 15

comment:5 by Jorma Rebane, 11 years ago

Keywords: reviewed added; review removed

Looks like a pretty harmless change in my opinion and can be committed to A15.

I'll mark the patch as 'reviewed' and it will be committed when feature lock is lifted.

Version 0, edited 11 years ago by Jorma Rebane (next)

comment:6 by Jorma Rebane, 11 years ago

Keywords: reviewed removed
Resolution: wontfix
Status: newclosed
Note: See TracTickets for help on using tickets.