Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Experiment with instruction level parallelism in pixelsearch1x.c #40

Open
wants to merge 497 commits into
base: master
Choose a base branch
from
Open

Experiment with instruction level parallelism in pixelsearch1x.c #40

wants to merge 497 commits into from

Conversation

wind0204
Copy link
Contributor

@wind0204 wind0204 commented Jan 2, 2024

Please see if this version has better performance than the non-parallel version if it interested you.

  • Dispatch 4 vector operations in each loop to allow a larger throughput in pixelsearch1x.c --I guess a CPU with decode width 5+ would accomplish the same throughput with just 2 vector operations per loop--
  • MOVMSKPS has twice the throughput of PMOVMSKB on AMD Zen2. --I guess it might help with the bottleneck on AMD Zen2--

Best regards.

@iseahound
Design Philosophy
92cf1da
@iseahound
Buffer now supports cropping and showing windows
8cd9144
@iseahound
Add setter to ImagePutBuffer
a1ada4e
@iseahound
Lift RGB colors to ARGB
50d9cfe
@iseahound
allow ImagePutBuffer().show to use window edges
222ffff
@iseahound
Update README.md
c4de443
@iseahound
C source files
28f49f5
@iseahound
C source files
02501e8
@iseahound
Add CPUID function
f221ac1
@iseahound
comment
eef2a0f
@iseahound
CPUID only uses 0-31
b35505d
@iseahound
Add pixelsearch with variation
8c8d2ce
@iseahound
C source files
70df42b
@iseahound
Fix colon : being removed from Drive path
ef0838b
@iseahound
Use the && short-circuit to simplify ternaries
f1f0a25
@iseahound
MimeType typo
1998b3d
@iseahound
Fix put_hex incorrect pointer to GlobalLock
7650cb1
@iseahound
Remove "smart_pointer" type from GdiplusShutdown
9d732c2
@iseahound
Change obj.pBitmap to pic.pBitmap
df74fcb
@iseahound
Remove unused parameter in GdiplusShutdown
d1d9225
@iseahound
remove annoying comments from select_
8537c07
@iseahound
WindowProc new thread should be critical
e39f101
@iseahound
Fix gdiplus remaining loaded with streams
3fb80f4
@iseahound
Try-Finally to avoid intermediate values
5cdf2d8
@iseahound
some comments
255fa3b
@iseahound
lowercase control flow
00ec0e2
@iseahound
better comment
d721a54
@iseahound
Prefix labels with ImagePut
91e5506
@iseahound
unnecessary double instantiation
33ddb7b
@iseahound
Switch case for Destroy()
b31e3c7
iseahound and others added 27 commits October 4, 2023 20:42
@iseahound
Improve ImageSearch
70f3372
@iseahound
Add count property
42f309b
@wind0204
Experiment with ImageSearchAll with tolerance to color variation
9ee8be9
@wind0204
Update ImageSearch method of BitmapBuffer
57cfafc
@wind0204
Remove an unnecessary assignment to a variable
d3c883d
@wind0204
Recover the lost work on my wife's windows system
dd7e974
@wind0204
Accelerate when the argument variation is 0
d834c70
@wind0204
Put back the carriage return characters on each line ending
05d7a47
@wind0204
As you learn English, the quality of the comments improves
8fd1cc0
@wind0204
Fix a bug where random memory location will be touched in focus-deter…
713c809 
…mine
@iseahound
Restore Default Behavior of ImageSearch
91a0e82
@iseahound
Fix ImageSearch2
4976903
@wind0204
Fix bug where focus search loop could go out of range
b9f6e19
@iseahound
Add files via upload
4161cc1 
Finalize ImageSearch1 code to be efficient and bug-free.
Supports transparency properly.
Fast!
@iseahound
C source files
365f9fa
@iseahound
Merge pull request iseahound#37 from wind0204/pr-fix_bug_where_focus_…
aad765f 
…search_loop_could_go_out_of_range

Fix bug where focus search loop could go out of range
@iseahound
C source files
d4e95cf
@iseahound
C source files
91c4d4b 
Rename pack to iter
@iseahound
C source files
714e15a 
Fix comment for pixelsearchall3
@iseahound
Finalize ImageSearch1
f20168f
@wind0204
Fix bug where the last scanline was skipped
c868eb2
@wind0204
The previous fix didn't fix, this one does seem to fix it
f50b1ba
@wind0204
The prev. commit had a bug of the possibility of reaching off haystack
2aa6843
@iseahound
Merge pull request iseahound#38 from wind0204/pr-fix_bug_where_last_s…
b27ac85 
…canline_was_skipped

Fix bug where the last scanline was skipped
@iseahound
comments: when the faeries dance in the wind
a552495
@wind0204
Merge branch 'iseahound:master' into master
346c611
@wind0204
Remove an unnecessary conditional jump
a0b9226
@wind0204 wind0204 mentioned this pull request Jan 2, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@wind0204 @iseahound @anonymous1184