<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
</head>
<body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">
Reviewed-by: Tim Rowley <<a href="mailto:timothy.o.rowley@intel.com" class="">timothy.o.rowley@intel.com</a>>
<div class=""><br class="">
<div>
<blockquote type="cite" class="">
<div class="">On Feb 3, 2017, at 11:35 AM, Bruce Cherniak <<a href="mailto:bruce.cherniak@intel.com" class="">bruce.cherniak@intel.com</a>> wrote:</div>
<br class="Apple-interchange-newline">
<div class="">
<div class="">Removed unused Clip() and FRUSTUM_CLIP_MASK define.<br class="">
---<br class="">
src/gallium/drivers/swr/rasterizer/core/clip.cpp | 22 ----------------------<br class="">
src/gallium/drivers/swr/rasterizer/core/clip.h   |  4 ----<br class="">
2 files changed, 26 deletions(-)<br class="">
<br class="">
diff --git a/src/gallium/drivers/swr/rasterizer/core/clip.cpp b/src/gallium/drivers/swr/rasterizer/core/clip.cpp<br class="">
index 7b1e09d..0a6afe5 100644<br class="">
--- a/src/gallium/drivers/swr/rasterizer/core/clip.cpp<br class="">
+++ b/src/gallium/drivers/swr/rasterizer/core/clip.cpp<br class="">
@@ -157,28 +157,6 @@ int ClipTriToPlane( const float *pInPts, int numInPts,<br class="">
    return i;<br class="">
}<br class="">
<br class="">
-<br class="">
-<br class="">
-void Clip(const float *pTriangle, const float *pAttribs, int numAttribs, float *pOutTriangles, int *numVerts, float *pOutAttribs)<br class="">
-{<br class="">
-    // temp storage to hold at least 6 sets of vertices, the max number that can be created during clipping<br class="">
-    OSALIGNSIMD(float) tempPts[6 * 4];<br class="">
-    OSALIGNSIMD(float) tempAttribs[6 * KNOB_NUM_ATTRIBUTES * 4];<br class="">
-<br class="">
-    // we opt to clip to viewport frustum to produce smaller triangles for rasterization precision<br class="">
-    int NumOutPts = ClipTriToPlane<FRUSTUM_NEAR>(pTriangle, 3, pAttribs, numAttribs, tempPts, tempAttribs);<br class="">
-    NumOutPts = ClipTriToPlane<FRUSTUM_FAR>(tempPts, NumOutPts, tempAttribs, numAttribs, pOutTriangles, pOutAttribs);<br class="">
-    NumOutPts = ClipTriToPlane<FRUSTUM_LEFT>(pOutTriangles, NumOutPts, pOutAttribs, numAttribs, tempPts, tempAttribs);<br class="">
-    NumOutPts = ClipTriToPlane<FRUSTUM_RIGHT>(tempPts, NumOutPts, tempAttribs, numAttribs, pOutTriangles, pOutAttribs);<br class="">
-    NumOutPts = ClipTriToPlane<FRUSTUM_BOTTOM>(pOutTriangles, NumOutPts, pOutAttribs, numAttribs, tempPts, tempAttribs);<br class="">
-    NumOutPts = ClipTriToPlane<FRUSTUM_TOP>(tempPts, NumOutPts, tempAttribs, numAttribs, pOutTriangles, pOutAttribs);<br class="">
-<br class="">
-    SWR_ASSERT(NumOutPts <= 6);<br class="">
-<br class="">
-    *numVerts = NumOutPts;<br class="">
-    return;<br class="">
-}<br class="">
-<br class="">
void ClipTriangles(DRAW_CONTEXT *pDC, PA_STATE& pa, uint32_t workerId, simdvector prims[], uint32_t primMask, simdscalari primId, simdscalari viewportIdx)<br class="">
{<br class="">
    SWR_CONTEXT *pContext = pDC->pContext;<br class="">
diff --git a/src/gallium/drivers/swr/rasterizer/core/clip.h b/src/gallium/drivers/swr/rasterizer/core/clip.h<br class="">
index f19858f..23a768f 100644<br class="">
--- a/src/gallium/drivers/swr/rasterizer/core/clip.h<br class="">
+++ b/src/gallium/drivers/swr/rasterizer/core/clip.h<br class="">
@@ -56,12 +56,8 @@ enum SWR_CLIPCODES<br class="">
    GUARDBAND_BOTTOM = (0x80 << CLIPCODE_SHIFT | 0x8)<br class="">
};<br class="">
<br class="">
-#define FRUSTUM_CLIP_MASK (FRUSTUM_LEFT|FRUSTUM_TOP|FRUSTUM_RIGHT|FRUSTUM_BOTTOM|FRUSTUM_NEAR|FRUSTUM_FAR)<br class="">
#define GUARDBAND_CLIP_MASK (FRUSTUM_NEAR|FRUSTUM_FAR|GUARDBAND_LEFT|GUARDBAND_TOP|GUARDBAND_RIGHT|GUARDBAND_BOTTOM|NEGW)<br class="">
<br class="">
-void Clip(const float *pTriangle, const float *pAttribs, int numAttribs, float *pOutTriangles,
<br class="">
-          int *numVerts, float *pOutAttribs);<br class="">
-<br class="">
INLINE<br class="">
void ComputeClipCodes(const API_STATE& state, const simdvector& vertex, simdscalar& clipCodes, simdscalari viewportIndexes)<br class="">
{<br class="">
-- <br class="">
2.7.4<br class="">
<br class="">
_______________________________________________<br class="">
mesa-dev mailing list<br class="">
<a href="mailto:mesa-dev@lists.freedesktop.org" class="">mesa-dev@lists.freedesktop.org</a><br class="">
https://lists.freedesktop.org/mailman/listinfo/mesa-dev<br class="">
</div>
</div>
</blockquote>
</div>
<br class="">
</div>
</body>
</html>