DynamoRIO / DynamoRIO/dynamorio

refactor cross-platform portions of core/x86/instrument.c

Open
#1,450 3 comments 0 reactions 0 assignees View on GitHub
Maintainability Migrated Priority-Medium Type-Feature
Dominant language
C
Stars
3.2k
Forks
629
Avg merge
2d 15h
Merged PRs (30d)
31

Description

_From [bruen...@google.com](https://code.google.com/u/109494838902877177630/) on May 28, 2014 11:23:48_

This is something that's long been on our clean-up list: move the general
client code out of core/x86/instrument.c into core/, or possibly into a new
subdir of core/ (core/client/, core/tool/, core/instrument/) if we'll have
more than one file. Would annotation support belong there? I don't think
so: annotations aren't strictly tied to tool instrumentation and are more
like the start/stop interface (consider "am I under DR?", DGC annotations,
etc.).

We could also consider putting cross-platform IR code into core/ir/.
This could include quite a bit of core/x86/instr.c (the code that just
operates on operand accessors), and core/instrlist.c.

_Original issue: http://code.google.com/p/dynamorio/issues/detail?id=1450_

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.