On Wed, 29 Jul 2026 08:01:26 GMT, Jayathirth D V <[email protected]> wrote:
> We are doing float->int conversion while populating vertex data for drawPoly > and drawParallelogram in MTLRenderer.fillVertex(). The final Vertex position > which is passed to shader also supports storing float data, so we should not > be doing this intermediate float to int conversion. This was noticed while > reviewing another PR: > https://github.com/openjdk/jdk/pull/31981#issuecomment-5045302871 > > This also deviates from what we do in OpenGL, where we directly store these > float values. We also add 0.5f values to these vertex position to hit pixel > centers for drawPoly, doing float->int conversion overrides this logic. > > Fix is to use float arguments for fillVertex. There are no regressions seen > in clientlibs testing. > Its is very difficult to write a reliable pass & fail regression test for > this fix. So i have added noreg-hard label in the bug. > > --------- > - [x] I confirm that I make this contribution in accordance with the [OpenJDK > Interim AI Policy](https://openjdk.org/legal/ai). LGTM ------------- Marked as reviewed by avu (Committer). PR Review: https://git.openjdk.org/jdk/pull/32083#pullrequestreview-4842204582
