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

port annotation infrastructure to AArchXX #1672

Open
derekbruening opened this issue Apr 9, 2015 · 3 comments
Open

port annotation infrastructure to AArchXX #1672

derekbruening opened this issue Apr 9, 2015 · 3 comments
Assignees
@derekbruening
Labels
help wanted OpSys-AArch64 OpSys-ARM Type-Feature

Comments

@derekbruening
Copy link
Contributor

Split from #1551
@derekbruening
Copy link
Contributor Author

We also need to port to AArch64 which we could include here as the implementations may be quite similar.

@derekbruening
Copy link
Contributor Author

There is further information here along with a potential contributor: https://groups.google.com/forum/#!topic/dynamorio-users/0HZGzngZmZ0

@derekbruening derekbruening mentioned this issue Jul 17, 2018
Open
@fhahn fhahn self-assigned this Jul 17, 2018
This was referenced Mar 7, 2020
Closed
Closed
derekbruening added a commit to DynamoRIO/drmemory that referenced this issue Mar 7, 2020
@derekbruening
DRi#4168: Annotations are not supported on ARM
54c3893 
DR does not support annotations on ARM (DRi#1672) so we have to
disable our annotations for non-x86.

Adds ARM and Android builds to Travis.

Issue: DynamoRIO/dynamorio#4168, DynamoRIO/dynamorio#1672
Fixes DynamoRIO/dynamorio#4168
@derekbruening derekbruening mentioned this issue Mar 7, 2020
Merged
derekbruening added a commit to DynamoRIO/drmemory that referenced this issue Mar 7, 2020
@derekbruening
DRi#4168: Annotations are not supported on ARM (#2274)
e7fdb70 
DR does not support annotations on ARM (DRi#1672) so we have to
disable our annotations for non-x86.

Adds ARM and Android builds to Travis.

Issue: DynamoRIO/dynamorio#4168, DynamoRIO/dynamorio#1672
Fixes DynamoRIO/dynamorio#4168
derekbruening added a commit to DynamoRIO/drmemory that referenced this issue Mar 21, 2020
@derekbruening
DRi#4168: Disable annotation header install for non-x86
5e69c84 
Fixes a broken install step for ARM by disabling the annotation header
install for non-x86.  DR does not yet support annotations on ARM
(DRi#1672).

Issue: DynamoRIO/dynamorio#4168, DynamoRIO/dynamorio#1672
@derekbruening derekbruening mentioned this issue Mar 21, 2020
Merged
derekbruening added a commit to DynamoRIO/drmemory that referenced this issue Mar 21, 2020
@derekbruening
DRi#4168: Disable annotation header install for non-x86 (#2287)
e647ea6 
Fixes a broken install step for ARM by disabling the annotation header
install for non-x86.  DR does not yet support annotations on ARM
(DRi#1672).

Issue: DynamoRIO/dynamorio#4168, DynamoRIO/dynamorio#1672
@derekbruening
Copy link
Contributor Author

@byron-hawkins made a nice parameterizable multi-threaded test app for annotations client.annotation-concurrency and we ended up using it as a multi-threaded non-trivially-sized test app for many drcachesim tests.
However, it contains annotations, so the lack of annotation support on AArchXX now blocks running many drcachesim tests. Ideally we would just implement annotation support on AArchXX, but we could potentially hack the test to make the annotation macros resolve to empty code to make progress.

@derekbruening derekbruening changed the title port annotation infrastructure to ARM port annotation infrastructure to AArchXX Oct 5, 2020
This was referenced Oct 6, 2020
Open
Closed
derekbruening added a commit that referenced this issue Oct 6, 2020
@derekbruening
i#4474 A64 testing: Enable drcachesim thread tests
bc9d80f 
Adds custom annotation macro defines to enable building the
client.annotation-concurrency executable and its shared library on
AArchXX.

Enables the 4 tests that require that executable, but do not need
annotation support: tool.drcachesim.threads,
tool.drcachesim.threads-with-config-file, tool.drcachesim.coherence,
and tool.drcachesim.TLB-threads.

Issue: #4474, #1672
@derekbruening derekbruening mentioned this issue Oct 6, 2020
Merged
derekbruening added a commit that referenced this issue Oct 6, 2020
@derekbruening
i#4474 A64 testing: Enable drcachesim thread tests (#4476)
352f990 
Adds custom annotation macro defines to enable building the
client.annotation-concurrency executable and its shared library on
AArchXX.

Enables the 4 tests that require that executable, but do not need
annotation support: tool.drcachesim.threads,
tool.drcachesim.threads-with-config-file, tool.drcachesim.coherence,
and tool.drcachesim.TLB-threads.

Issue: #4474, #1672
@derekbruening derekbruening mentioned this issue Mar 29, 2021
Open
@derekbruening derekbruening mentioned this issue Sep 14, 2022
Closed
@derekbruening derekbruening assigned derekbruening and unassigned fhahn Jan 25, 2023
@derekbruening derekbruening mentioned this issue Oct 17, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@derekbruening derekbruening

Labels
help wanted OpSys-AArch64 OpSys-ARM Type-Feature
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@fhahn @byron-hawkins @derekbruening