| Commit message (Collapse) | Author | Age |
|
|
|
|
|
|
| |
http://codereview.appspot.com/4548090/
git-svn-id: http://skia.googlecode.com/svn/trunk@1534 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
|
|
| |
git-svn-id: http://skia.googlecode.com/svn/trunk@1533 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
|
|
|
|
|
|
| |
Review URL: http://codereview.appspot.com/4591044/
git-svn-id: http://skia.googlecode.com/svn/trunk@1532 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
|
|
| |
git-svn-id: http://skia.googlecode.com/svn/trunk@1531 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
|
|
|
|
|
|
| |
http://code.google.com/p/chromium/issues/detail?id=83934
git-svn-id: http://skia.googlecode.com/svn/trunk@1528 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
|
|
| |
git-svn-id: http://skia.googlecode.com/svn/trunk@1527 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
see http://codereview.appspot.com/4580043
1. Create a single public skia.gyp file that all outside projects (Chrome,
Android, etc.) should depend on from now on. I haven't yet created targets
suitable for those projects to use, but this is where we should add them.
2. Make gyp generate its Makefiles within out/ directory, rather than polluting
directories under source control.
3. Modify trunk/Makefile to automatically run gyp_skia and then call the
generated Makefile, to ease developer transition.
git-svn-id: http://skia.googlecode.com/svn/trunk@1526 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
|
|
|
|
|
|
|
|
| |
floating point vs. double matrices.
Over-the-shoulder review by reed@.
git-svn-id: http://skia.googlecode.com/svn/trunk@1525 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
|
|
| |
git-svn-id: http://skia.googlecode.com/svn/trunk@1524 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
|
|
|
|
|
| |
Patch by Ryan Sleevi <rsleevi@chromium.org>
Review URL: http://codereview.appspot.com/4568055/
git-svn-id: http://skia.googlecode.com/svn/trunk@1523 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Fix for putImageData
TEST=manually ran tests that were failing
BUG=84165
TBR=reed@google.com
Code review URL: http://codereview.appspot.com/4570042
git-svn-id: http://skia.googlecode.com/svn/trunk@1518 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
|
|
|
|
|
|
| |
Unifies SkMatrix::hasPerspective(), has_perspective(), and manual tests of SkMatrix::getType & kPerspective_Mask.
git-svn-id: http://skia.googlecode.com/svn/trunk@1517 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
|
|
| |
git-svn-id: http://skia.googlecode.com/svn/trunk@1516 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
|
|
| |
git-svn-id: http://skia.googlecode.com/svn/trunk@1515 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
|
|
| |
git-svn-id: http://skia.googlecode.com/svn/trunk@1514 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
|
|
| |
git-svn-id: http://skia.googlecode.com/svn/trunk@1513 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
with direct floating-point comparisons in computeTypeMask() is a significant win
(up to 40%?).
In practice, we see a 10-15% speedup in this function (the current most
expensive Skia function) in Chrome on the FishIETank demo.
The remainder of the function looks like it ought to be amenable to similar
speedups, but we haven't found a rewrite that behaves well in benchmarking yet.
http://codereview.appspot.com/4551097/
git-svn-id: http://skia.googlecode.com/svn/trunk@1512 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
|
|
|
|
|
|
|
|
| |
operations
http://http://codereview.appspot.com/4532116/
git-svn-id: http://skia.googlecode.com/svn/trunk@1511 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
|
|
| |
git-svn-id: http://skia.googlecode.com/svn/trunk@1510 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
|
|
|
|
|
|
| |
init our float/double arrays, so we get reliable timings (NaNs are slow)
git-svn-id: http://skia.googlecode.com/svn/trunk@1509 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
|
|
|
|
|
|
| |
third_party/gyp
git-svn-id: http://skia.googlecode.com/svn/trunk@1506 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
|
|
| |
git-svn-id: http://skia.googlecode.com/svn/trunk@1505 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
|
|
| |
git-svn-id: http://skia.googlecode.com/svn/trunk@1504 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
|
|
| |
git-svn-id: http://skia.googlecode.com/svn/trunk@1503 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
|
|
| |
git-svn-id: http://skia.googlecode.com/svn/trunk@1502 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
|
|
| |
git-svn-id: http://skia.googlecode.com/svn/trunk@1501 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
|
|
|
|
|
|
|
|
|
| |
- data on heap instead of stack
- more closely match operations of actual SkMatrix::setConcat()
- avoid random perturbations, do_always;
use perf annotate to verify that assembly isn't getting optimized away
git-svn-id: http://skia.googlecode.com/svn/trunk@1500 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
|
|
| |
git-svn-id: http://skia.googlecode.com/svn/trunk@1499 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
|
|
|
|
|
|
| |
default is still 0 for the moment.
git-svn-id: http://skia.googlecode.com/svn/trunk@1498 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
<!-- Begin help window -->
<script type="text/javascript"><!--
var xsrfToken = 'dd5d1a57ca1d5440593050fdc596d377';
var helpDisplayed = false;
document.onclick = M_clickCommon;
var media_url = "/static/";
var base_url = "/";
var publish_link = "/4571045/publish";
// -->
</script>
<div id="help" style="display: none;">
<div style="font-size: medium; text-align: center;">Keyboard Shortcuts</div>
<hr>
<table width="100%">
<tbody><tr valign="top">
<td>
<table width="100%">
<tbody><tr>
<td></td><th>File</th>
</tr>
<tr>
<td class="shortcut"><span class="letter">u</span> <b>:</b></td><td>up to issue</td>
</tr>
<tr>
<td class="shortcut"><span class="letter">m</span> <b>:</b></td><td>publish + mail comments</td>
</tr>
<tr>
<td class="shortcut"><span class="letter">M</span> <b>:</b></td><td>edit review message</td>
</tr>
<tr>
<td class="shortcut"><span class="letter">j</span> / <span class="letter">k</span> <b>:</b></td><td>jump to file after / before current file</td>
</tr>
<tr>
<td class="shortcut"><span class="letter">J</span> / <span class="letter">K</span> <b>:</b></td><td>jump to next file with a comment after / before current file</td>
</tr>
<tr>
<td></td><th>Side-by-side diff</th>
</tr>
<tr>
<td class="shortcut"><span class="letter">i</span> <b>:</b></td><td>toggle intra-line diffs</td>
</tr>
<tr>
<td class="shortcut"><span class="letter">e</span> <b>:</b></td><td>expand all comments</td>
</tr>
<tr>
<td class="shortcut"><span class="letter">c</span> <b>:</b></td><td>collapse all comments</td>
</tr>
<tr>
<td class="shortcut"><span class="letter">s</span> <b>:</b></td><td>toggle showing all comments</td>
</tr>
<tr>
<td class="shortcut"><span class="letter">n</span> / <span class="letter">p</span> <b>:</b></td><td>next / previous diff chunk or comment</td>
</tr>
<tr>
<td class="shortcut"><span class="letter">N</span> / <span class="letter">P</span> <b>:</b></td><td>next / previous comment</td>
</tr>
<tr>
<td class="shortcut"><span class="letter"><Enter></span> <b>:</b></td><td>respond to / edit current comment</td>
</tr>
</tbody></table>
</td>
<td>
<table width="100%">
<tbody><tr>
<td></td><th>Issue</th>
</tr>
<tr>
<td class="shortcut"><span class="letter">u</span> <b>:</b></td><td>up to list of issues</td>
</tr>
<tr>
<td class="shortcut"><span class="letter">m</span> <b>:</b></td><td>publish + mail comments</td>
</tr>
<tr>
<td class="shortcut"><span class="letter">j</span> / <span class="letter">k</span> <b>:</b></td><td>jump to patch after / before current patch</td>
</tr>
<tr>
<td class="shortcut"><span class="letter">o</span> / <span class="letter"><Enter></span> <b>:</b></td><td>open current patch in side-by-side view</td>
</tr>
<tr>
<td class="shortcut"><span class="letter">i</span> <b>:</b></td><td>open current patch in unified diff view</td>
</tr>
<tr><td> </td></tr>
<tr><td></td><th>Issue List</th></tr>
<tr>
<td class="shortcut"><span class="letter">j</span> / <span class="letter">k</span> <b>:</b></td><td>jump to issue after / before current issue</td>
</tr>
<tr>
<td class="shortcut"><span class="letter">o</span> / <span class="letter"><Enter></span> <b>:</b></td><td>open current issue</td>
</tr>
<tr>
<td class="shortcut"><span class="letter">#</span> <b>:</b></td>
<td>close issue</td>
</tr>
<tr><td> </td></tr>
<tr>
<td></td><th>Comment/message editing</th>
</tr>
<tr>
<td class="shortcut"><span class="letter"><Ctrl></span> + <span class="letter">s</span> <b>:</b></td><td>save comment</td>
</tr>
<tr>
<td class="shortcut"><span class="letter"><Esc></span> <b>:</b></td><td>cancel edit</td>
</tr>
</tbody></table>
</td>
</tr>
</tbody></table>
</div>
<!-- End help window -->
<div align="right">
<div style="float:left; font-weight:bold"><i>Rietveld</i> Code Review Tool</div>
<b>tomhudson@google.com (TomH)</b>
|
<a class="novisit" href="/settings">Settings</a>
|
<a class="novisit" target="_blank" href="http://code.google.com/p/rietveld/wiki/CodeReviewHelp">Help</a>
|
<a class="novisit" target="_blank" href="http://code.google.com/p/rietveld/issues/list">Bug tracker</a>
|
<a class="novisit" target="_blank" href="http://groups.google.com/group/codereview-discuss">Discussion group</a>
|
<a class="novisit" target="_blank" href="http://code.google.com/p/rietveld">Source code</a>
|
<a class="novisit" href="http://codereview.appspot.com/_ah/logout?continue=https://www.google.com/accounts/Logout%3Fcontinue%3Dhttps://appengine.google.com/_ah/logout%253Fcontinue%253Dhttp://codereview.appspot.com/4571045/%26service%3Dah">Sign out</a>
</div>
<div class="counter">(691)</div>
<div class="mainmenu">
<a href="/" class="active">Issues</a>
<a href="/repos">Repositories</a>
<a href="/search">Search</a>
</div>
<div class="mainmenu2">
<a href="/use_uploadpy">Create Issue</a>
|
<a href="/mine">My Issues</a>
|
<a href="/starred">Starred</a>
|
<a href="/all">Open</a>
|
<a href="/all?closed=1">All</a>
</div>
<div>
<h2>
<span id="issue-star-4571045">
<a href="javascript:M_addIssueStar(4571045)">
<img src="/static/star-dark.gif" width="15" height="15" border="0"></a>
</span>
<span class="issue-close" id="issue-close-4571045">
<a href="javascript:M_closeIssue(4571045)">
<img src="/static/close.gif" title="Close This Issue" width="15" height="15" border="0"></a>
</span>
Issue <a href="/4571045/" onmouseover="M_showPopUp(this, 'popup-issue');">
4571045</a>:
New matrix benchmarks to evaluate ::setConcat implementation options
</h2>
<table class="issue-details" border="0" width="100%">
<tbody><tr valign="top">
<td class="meta" width="20%">
<div>
<a class="novisit" href="/4571045/edit">
Edit Issue
</a>
<br>
<a class="novisit" href="/4571045/publish">
Publish+Mail Comments
</a> ('m')
<br>
<a class="novisit" href="/4571045/diff/1/bench/MatrixBench.cpp">
<b>Start Review</b>
</a>
</div>
<div class="issue_details_sidebar">
<div><b>Created:</b><br>
16 minutes ago by me
</div>
<div><b>Modified:</b><br>
0 minutes ago
</div>
<div><b>Reviewers:</b><br>
<a href="/user/reed1" onmouseover="M_showUserInfoPopup(this)">reed1</a>
</div>
<div><b>CC:</b><br>
</div>
<div><b>Base URL:</b><br>
http://skia.googlecode.com/svn/trunk/
</div>
<div><b>Visibility:</b><br>
Public.
</div>
</div>
</td>
<td style="padding-left: .8em; padding-right: .8em;" width="80%">
<h3><a id="issue-description-pointer" href="javascript:M_toggleSection('issue-description')" class="toggled-section opentriangle">
Description</a></h3>
<div id="issue-description" style="margin-left:15px;">
<pre>On platforms that use Float (instead of Fixed), SkMatrix stores its internal
values as floats, but setConcat() needs extra precision and so uses doubles to
contain intermediate values.
These three benchmarks compare the speed of float-only, double-only, and
float-cast-to-double implementations of a chunk of code extracted from the
non-perspective case of setConcat().
</pre>
</div>
<h3>
<a id="ps-1-pointer" href="javascript:M_toggleSectionForPS('4571045', '1')" class="toggled-section opentriangle">
Patch Set 1
</a>
</h3>
<div id="ps-1" style="">
<div class="issue-list">
<div class="pagination">
<div style="float: left;">
<i>Created:</i> 16 minutes ago
</div>
<div style="float: right;">
<a href="/download/issue4571045_1.diff">
Download raw patch set</a>
</div>
<div style="clear:both;"></div>
</div>
<table id="queues" style="clear:both;">
<tbody><tr align="left">
<th colspan="2"></th>
<th>Unified diffs</th>
<th>Side-by-side diffs</th>
<th>Delta from patch set</th>
<th colspan="3">Stats</th>
<th>Patch</th>
</tr>
<tr name="patch">
<td class="first" width="14"><img src="/static/closedtriangle.gif" style="" width="12" height="9"></td>
<td style="white-space: nowrap">M </td>
<td>
<a class="noul" href="/4571045/patch/1/2">
bench/MatrixBench.cpp
</a>
</td>
<td>
<a class="noul" href="/4571045/diff/1/bench/MatrixBench.cpp">
View
</a>
</td>
<td style="white-space: nowrap">
</td>
<td style="white-space: nowrap">4 chunks</td>
<td style="white-space: nowrap">+114 lines, -3 lines</td>
<td style="white-space: nowrap">
0 comments
</td>
<td>
<a href="/download/issue4571045_1_2.diff" title="Download patch for bench/MatrixBench.cpp">
Download
</a>
</td>
</tr>
</tbody></table>
</div>
</div>
<h3>
<a id="messages-pointer" href="javascript:M_toggleSection('messages')" class="toggled-section opentriangle">
Messages
</a>
</h3>
<div><i>Total messages: 3</i></div>
<div id="messages">
<div style="margin-bottom: .5em;">
<a href="javascript:M_showAllComments('cl', 3)">
Expand All Messages</a>
|
<a href="javascript:M_hideAllComments('cl', 3)">
Collapse All Messages</a>
</div>
<div class="message " id="msg-agpjb2RlcmV2aWV3chwLEgVJc3N1ZRil_5YCDAsSB01lc3NhZ2UY6QcM" name="0">
<a name="msg-agpjb2RlcmV2aWV3chwLEgVJc3N1ZRil_5YCDAsSB01lc3NhZ2UY6QcM"></a>
<div class="header">
<table border="0" width="100%" cellspacing="0" cellpadding="0">
<tbody><tr class="comment_title" onclick="M_switchChangelistComment(0)">
<td style="padding-left: 5px; white-space: nowrap;">
<b>me</b>
</td>
<td width="100%">
<table style="table-layout:fixed; white-space: nowrap;" width="100%">
<tbody><tr>
<td>
<div style="white-space: nowrap; overflow: hidden;" class="extra" id="cl-preview-0">
</div>
</td>
</tr>
</tbody></table>
</td>
<td align="right" style="white-space: nowrap; padding-right: 5px;">
13 minutes ago
</td>
</tr>
</tbody></table>
</div>
<div id="cl-comment-0" style="display: none;">
<div class="message-body">
<pre name="cl-message-0"></pre>
</div>
<div class="message-actions">
<a href="javascript:M_replyToMessage('0', '2011/06/03 18:57:35', 'TomH')" id="message-reply-href-0">Reply</a>
<textarea rows="7" cols="70" name="message" style="display:none"></textarea>
<div id="message-reply-0" style="display:none;"></div>
</div>
</div>
</div>
<div class="message " id="msg-agpjb2RlcmV2aWV3chwLEgVJc3N1ZRil_5YCDAsSB01lc3NhZ2UY0Q8M" name="1">
<a name="msg-agpjb2RlcmV2aWV3chwLEgVJc3N1ZRil_5YCDAsSB01lc3NhZ2UY0Q8M"></a>
<div class="header">
<table border="0" width="100%" cellspacing="0" cellpadding="0">
<tbody><tr class="comment_title" onclick="M_switchChangelistComment(1)">
<td style="padding-left: 5px; white-space: nowrap;">
<b>reed1</b>
</td>
<td width="100%">
<table style="table-layout:fixed; white-space: nowrap;" width="100%">
<tbody><tr>
<td>
<div style="white-space: nowrap; overflow: hidden;" class="extra" id="cl-preview-1">
1. lets remove the "fix" from the function name. just muladdmul I think 2. we ...
</div>
</td>
</tr>
</tbody></table>
</td>
<td align="right" style="white-space: nowrap; padding-right: 5px;">
10 minutes ago
</td>
</tr>
</tbody></table>
</div>
<div id="cl-comment-1" style="display: none;">
<div class="message-body">
<pre name="cl-message-1">1. lets remove the "fix" from the function name. just muladdmul I think
2. we can remove the return, since it will be compiled away anyway
3. nice checkin comment!
4. what are the results like?</pre>
</div>
<div class="message-actions">
<a href="javascript:M_replyToMessage('1', '2011/06/03 19:00:39', 'reed1')" id="message-reply-href-1">Reply</a>
<textarea rows="7" cols="70" name="message" style="display:none"></textarea>
<div id="message-reply-1" style="display:none;"></div>
</div>
</div>
</div>
<div class="message " id="msg-agpjb2RlcmV2aWV3chwLEgVJc3N1ZRil_5YCDAsSB01lc3NhZ2UYuRcM" name="2">
<a name="msg-agpjb2RlcmV2aWV3chwLEgVJc3N1ZRil_5YCDAsSB01lc3NhZ2UYuRcM"></a>
<div class="header">
<table border="0" width="100%" cellspacing="0" cellpadding="0">
<tbody><tr class="comment_title" onclick="M_switchChangelistComment(2)">
<td style="padding-left: 5px; white-space: nowrap;">
<b>me</b>
</td>
<td width="100%">
<table style="table-layout:fixed; white-space: nowrap;" width="100%">
<tbody><tr>
<td>
<div style="white-space: nowrap; overflow: hidden; visibility: hidden;" class="extra" id="cl-preview-2">
> 4. what are the results like? For posterity: tomhudson@tomhudson-zx600-linux:/usr/local/google/src/skia3$ out/bench/bench -match matrix_concat -repeat 1000 ...
</div>
</td>
</tr>
</tbody></table>
</td>
<td align="right" style="white-space: nowrap; padding-right: 5px;">
0 minutes ago
</td>
</tr>
</tbody></table>
</div>
<div id="cl-comment-2">
<div class="message-body">
<pre name="cl-message-2">> 4. what are the results like?
For posterity:
tomhudson@tomhudson-zx600-linux:/usr/local/google/src/skia3$ out/bench/bench
-match matrix_concat -repeat 1000
skia bench: alpha=0xFF antialias=1 filter=0
running bench [640 480] matrix_concat_double 8888: msecs = 0.79 565:
msecs = 0.79 GPU: msecs = 0.79
running bench [640 480] matrix_concat_floatdouble 8888: msecs = 0.97 565:
msecs = 0.97 GPU: msecs = 0.97
running bench [640 480] matrix_concat_float 8888: msecs = 0.74 565:
msecs = 0.73 GPU: msecs = 0.74</pre>
</div>
<div class="message-actions">
<a href="javascript:M_replyToMessage('2', '2011/06/03 19:11:06', 'TomH')" id="message-reply-href-2">Reply</a>
<textarea rows="7" cols="70" name="message" style="display:none"></textarea>
<div id="message-reply-2" style="display:none;"></div>
</div>
</div>
</div>
<div>
<a href="javascript:M_showAllComments('cl', 3)">
Expand All Messages</a>
|
<a href="javascript:M_hideAllComments('cl', 3)">
Collapse All Messages</a>
</div>
</div>
<script language="JavaScript" type="text/javascript">
<!--
document.onkeydown = M_changelistKeyDown;
var dashboardState = new M_DashboardState(window, 'patch', 'M_CLPatchMarker');
M_switchChangelistCommentByAnchor();
// -->
</script>
<div style="display:none;">
<form method="POST" action="/4571045/publish" id="message-reply-form">
<input type="hidden" name="xsrf_token" value="dd5d1a57ca1d5440593050fdc596d377">
<div></div>
<input type="hidden" name="subject" value="New matrix benchmarks to evaluate ::setConcat implementation options">
<input type="hidden" name="message_only" value="1">
<input type="submit" value="Send Message">
<input type="button" value="Discard" name="discard">
<input type="checkbox" name="send_mail" value="1" id="message-reply-send-mail" checked="checked">
<label>Send mail to reviewers</label>
</form>
</div>
<a id="resizer" class="resizer" style="display:none;cursor:pointer">
<img src="/static/zippyplus.gif">
</a>
</td>
</tr>
</tbody></table>
</div>
<div class="popup" id="popup-issue" style="left: 104px; top: 116px; visibility: hidden; ">
<b>Issue 4571045: New matrix benchmarks to evaluate ::setConcat implementation options
</b><br>
Created 16 minutes ago by me<br>
Modified 0 minutes ago<br>
Reviewers: reed1<br>
Base URL: http://skia.googlecode.com/svn/trunk/<br>
Comments: 0
</div>
<p></p>
<div style="float: left;">
<a target="_blank" href="http://code.google.com/appengine/"><img border="0" src="/static/appengine-noborder-120x30.gif" alt="Powered by Google App Engine"></a>
</div>
<div class="extra" style="font-size: 9pt; float: right; text-align: right;">
<div style="height:14px;">
<img src="/static/rss.gif" alt="RSS Feeds" width="14" height="14" align="top">
<a href="/rss/all">Recent Issues</a>
|
<a href="/rss/mine/TomH">My Issues</a>
|
<a href="/rss/reviews/TomH">My Reviews</a>
|
<a href="/rss/closed/TomH">My Closed</a>
|
<a href="/rss/issue/4571045">This issue</a>
</div>
<div style="margin-top: .3em;">This is Rietveld <a href="http://code.google.com/p/rietveld/updates/list">r756
</a></div>
</div>
<script type="text/javascript">
var gaJsHost = (("https:" == document.location.protocol) ? "https://ssl." : "http://www.");
document.write(unescape("%3Cscript src='" + gaJsHost + "google-analytics.com/ga.js' type='text/javascript'%3E%3C/script%3E"));
</script><script src="http://www.google-analytics.com/ga.js" type="text/javascript"></script>
<script type="text/javascript">
var pageTracker = _gat._getTracker("UA-4803694-4");
pageTracker._initData();
pageTracker._trackPageview();
</script>
On platforms that use Float (instead of Fixed), SkMatrix stores its internal
values as floats, but setConcat() needs extra precision and so uses doubles
to contain intermediate values.
These three benchmarks compare the speed of float-only, double-only, and
float-cast-to-double implementations of a chunk of code extracted from the
non-perspective case of setConcat().
git-svn-id: http://skia.googlecode.com/svn/trunk@1497 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
|
|
|
|
|
|
| |
see http://codereview.appspot.com/4571044/
git-svn-id: http://skia.googlecode.com/svn/trunk@1494 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
|
|
|
|
|
|
| |
see http://codereview.appspot.com/4530104
git-svn-id: http://skia.googlecode.com/svn/trunk@1490 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
|
|
|
|
|
| |
http://codereview.appspot.com/4568052/
git-svn-id: http://skia.googlecode.com/svn/trunk@1488 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
|
|
|
|
| |
Review=http://codereview.appspot.com/4517126
git-svn-id: http://skia.googlecode.com/svn/trunk@1483 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
|
|
|
|
|
|
| |
see http://codereview.appspot.com/4515181/
git-svn-id: http://skia.googlecode.com/svn/trunk@1482 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
|
|
|
|
|
| |
http://codereview.appspot.com/4517133/
git-svn-id: http://skia.googlecode.com/svn/trunk@1481 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
|
|
|
|
|
|
| |
http://codereview.appspot.com/4561058.
git-svn-id: http://skia.googlecode.com/svn/trunk@1480 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
|
|
|
|
|
|
| |
see http://codereview.appspot.com/4536096/
git-svn-id: http://skia.googlecode.com/svn/trunk@1479 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
|
|
| |
git-svn-id: http://skia.googlecode.com/svn/trunk@1478 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
|
|
| |
git-svn-id: http://skia.googlecode.com/svn/trunk@1477 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
utils files from the build of views.a, since they're already being compiled into
utils.a, since it was causing unnecessary rebuilds (the same could probably be
done for linux and mac). Remove the commented-out SkImageDecoder_libpvjpeg.cpp,
which has been removed from SVN.
http://codereview.appspot.com/4532105/
git-svn-id: http://skia.googlecode.com/svn/trunk@1476 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
|
|
|
|
|
|
| |
http://codereview.appspot.com/4536100/
git-svn-id: http://skia.googlecode.com/svn/trunk@1475 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
|
|
| |
git-svn-id: http://skia.googlecode.com/svn/trunk@1474 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
|
|
| |
git-svn-id: http://skia.googlecode.com/svn/trunk@1473 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
|
|
|
|
|
|
| |
function, do 32b at a time instead of 8b at a time.
git-svn-id: http://skia.googlecode.com/svn/trunk@1472 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
|
|
| |
git-svn-id: http://skia.googlecode.com/svn/trunk@1471 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
|
|
|
|
|
|
| |
Review URL: http://codereview.appspot.com/4536098/
git-svn-id: http://skia.googlecode.com/svn/trunk@1470 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
|
|
|
|
|
| |
http://codereview.appspot.com/4564052/
git-svn-id: http://skia.googlecode.com/svn/trunk@1469 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
|
|
| |
git-svn-id: http://skia.googlecode.com/svn/trunk@1468 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
|
|
|
|
|
| |
http://codereview.appspot.com/4551083/
git-svn-id: http://skia.googlecode.com/svn/trunk@1467 2bbb7eff-a529-9590-31e7-b0007b416f81
|