Cleaned dead/debug code in X86BinaryAssembler, all tests pass

[opencode-agent]

The review of X86BinaryAssembler.java has been completed with the following fixes applied:

Changes Made:

1. Line 627 - Removed debug System.out.println statement in getUnresolvedObjectRefs()

2. Lines 2271-2276 - Simplified redundant exception handling in writeJECXZ():

   • The original code threw IllegalArgumentException but caught UnresolvedObjectRefException (which was never thrown), resulting in unreachable catch block
   • Now directly throws IllegalArgumentException

3. Lines 2458-2517 - Cleaned up dead code in writeLEA() method:

   • Removed if (true) block with internal comments indicating incomplete refactoring
   • Removed dead code paths with && false conditions
   • Removed duplicate ModRM encoding logic that was never executed
   • The method now correctly uses the proper helper methods (writeModRMRREXPrefix, write8)

Build Status: BUILD SUCCESSFUL - All tests pass after changes.

Closes #417

opencode session  |  github run