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

Issue 288720043: Fixes drcachesim.multiproc test failure on ARM machine

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

Description

Commit log for first patchset: --------------- Fixes drcachesim.multiproc test failure on ARM machine - handle prefetch hits/misses in drcachesim.multiproc output ---------------

Patch Set 1 #

Patch Set 2 : update commit msg #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M clients/drcachesim/tests/multiproc.templatex View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 3
zhaoqin
8 years, 3 months ago (2016-01-12 02:34:41 UTC) #1
zhaoqin
Commit log for latest patchset: --------------- Fixes drcachesim.multiproc test failure on ARM machines - handle ...
8 years, 3 months ago (2016-01-12 02:35:48 UTC) #2
bruening
8 years, 3 months ago (2016-01-12 05:44:08 UTC) #3
> Commit log for latest patchset:
> ---------------
> Fixes drcachesim.multiproc test failure on ARM machines

The title line is always the unconjugated infinitive form: it should be "Fix"
not "Fixes".

Else LGTM
Sign in to reply to this message.

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