DynamoRIO / DynamoRIO/dynamorio

Can we have adaptive peephole optimization back?

Open
#4,483 2 comments 0 reactions 0 assignees View on GitHub
Type-Feature
Dominant language
C
Stars
3.2k
Forks
629
Avg merge
2d 15h
Merged PRs (30d)
31

Description

Hi devs, I believe that we should have adaptive peephole optimization back. I believe that this should not be combined with -opt_speed, but should be a separate option, -opt_peephole, or should be implemented as a client. I believe that peephole optimization should always be performed after the -opt_speed inlining, if -opt_speed is passed, since we want to follow the libminecraft inline-then-optimize concept. Also, if you guys over-estimate compiler optimizations, then it's also possible for clients to generate inefficient code as well, and inlining opens up more opportunities for peephole optimization (e.g how about register allocation/stack optimization/whatever you call it after you inlined the unconditional branch/method call?).

[here's a link to a video about libminecraft's advanced-technology whole-program optimization](https://www.youtube.com/watch?v=tONR_7L7x80)

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.