Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(362)

Issue 252600043: TBR: i#1732 drcpusim: add tests for different CPU models

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 11 months ago by bruening
Modified:
10 years, 11 months ago
Reviewers:
zhaoqin
CC:
dynamorio-devs_googlegroups.com
Visibility:
Public.

Description

Commit log for first patchset: --------------- i#1732 drcpusim: add tests for different CPU models Adds 8 new tests: one sanity check of simple_app and 7 targeted asm tests for Pentium through Penryn. The asm tests are Linux-only as it's much easier to have a pure-static app for these. ---------------

Patch Set 1 #

Patch Set 2 : Committed #

Unified diffs Side-by-side diffs Delta from patch set Stats (+168 lines, -92 lines) Patch
A + clients/drcpusim/tests/mmx.asm View 2 chunks +10 lines, -11 lines 0 comments Download
A clients/drcpusim/tests/mmx.templatex View 1 chunk +1 line, -0 lines 0 comments Download
A + clients/drcpusim/tests/sse.asm View 2 chunks +12 lines, -11 lines 0 comments Download
A clients/drcpusim/tests/sse.templatex View 1 chunk +1 line, -0 lines 0 comments Download
A + clients/drcpusim/tests/sse2.asm View 2 chunks +14 lines, -11 lines 0 comments Download
A clients/drcpusim/tests/sse2.templatex View 1 chunk +1 line, -0 lines 0 comments Download
A + clients/drcpusim/tests/sse3.asm View 2 chunks +16 lines, -11 lines 0 comments Download
A clients/drcpusim/tests/sse3.templatex View 1 chunk +1 line, -0 lines 0 comments Download
A + clients/drcpusim/tests/sse41.asm View 2 chunks +24 lines, -11 lines 0 comments Download
A clients/drcpusim/tests/sse41.templatex View 1 chunk +1 line, -0 lines 0 comments Download
A + clients/drcpusim/tests/sse42.asm View 2 chunks +26 lines, -26 lines 0 comments Download
A clients/drcpusim/tests/sse42.templatex View 1 chunk +1 line, -0 lines 0 comments Download
A + clients/drcpusim/tests/ssse3.asm View 2 chunks +22 lines, -11 lines 0 comments Download
A clients/drcpusim/tests/ssse3.templatex View 1 chunk +1 line, -0 lines 0 comments Download
M core/arch/asm_defines.asm View 1 chunk +2 lines, -0 lines 0 comments Download
M suite/tests/CMakeLists.txt View 1 chunk +35 lines, -0 lines 0 comments Download

Messages

Total messages: 2
bruening
10 years, 11 months ago (2015-07-30 23:06:45 UTC) #1
bruening
10 years, 11 months ago (2015-07-30 23:06:48 UTC) #2
Committed as
https://github.com/DynamoRIO/dynamorio/commit/b756d36f5cc0e3a87eff009df840427...

Final commit log: 
---------------
i#1732 drcpusim: add tests for different CPU models

Adds 8 new tests: one sanity check of simple_app and 7 targeted asm tests
for Pentium through Penryn.  The asm tests are Linux-only as it's much
easier to have a pure-static app for these.

Review-URL: https://codereview.appspot.com/252600043
---------------
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b