calls to static interface methods are broken for primitive args

Description

Same issue as CLJ-2284, but the committed fix didn't handle the primitive path.

Proposed: Handle both primitive and non-primitive cases by making a more generic static initialization in asm, rather than the convenience gen.invokeStatic() call.

Patch: clj-2517-2.patch

Screened by: Alex Miller

Environment

None

Activity

Show:

Alex Miller

August 22, 2019, 7:48 PM

Copy link to comment

Added clj-2517-2.patch which just changes spaces to tabs in the Compiler change to reduce whitespace diffs. No semantic changes, attribution retained.

Alexander Yakushev

February 28, 2020, 2:51 PM

Edited

Copy link to comment

Bump. Ran hard into this today, basically retraced all steps from discovering 2284, fixing the bug in emitUnboxed, and only then finding this ticket. It would be nice if this is included in 1.11, for now we have to run a fork to upgrade to JDK11.

Alex Miller

February 28, 2020, 2:57 PM

Copy link to comment

It's at the top of the pile, so for sure.

Alex Miller

March 3, 2020, 7:20 PM

Copy link to comment

Applied for 1.10.2