Allow stereo rendering patches to run without DLL for debugging

Due to RenderDoc and Nsight Graphics not really being compatible with large .DLL projects like BetterVR, you normally aren't able to use a graphical debugger to debug issues like shadows for example.

With these changes, you can make a lite version of the mod that does all the stereo rendering things, but without the DLL, allowing a developer to still use RenderDoc to debug certain issues that just simply are way easier when you can see the intermediary drawing results of the game. To use this lite version, you need to copy these files into a separate graphic pack (or symlink them):
- patch_PatchUtils.asm
- patch_RND_Logging.asm
- patch_RND_StereoRendering.asm
- patch_RND_StereoRendering_ActorJobs.asm
- patch_RND_StereoRendering_CameraAndProjection.asm
- patch_STUB_StereoRenderingDebugging.asm
- and of course a rules.txt file, but you should probably modify it.

Then, if you switch the "useStubHooks" value from .int 0 to .int 1, you'll be able to go in-game with the appropriate performance impact that VR will have.

The goal definitely isn't to make ALL patches run without VR, just enough that I can debug stereo rendering issues or issues related to the camera modifications and projection matrices used for VR.

Author: Crementif

resources/BreathOfTheWild_BetterVR/patch_RND_StereoRendering_ActorJobs.asm

@@ -3,10 +3,6 @@ moduleMatches = 0x6267BFD0
 
 .origin = codecave
 
-
-
-
-
 0x037A6EC4 = real_actor_job1_1:
 0x101E5FFC = Player_vtable:
 
@@ -496,6 +492,29 @@ scopedDeltaSetter_float2:
 strActor_job0_1:
 .string "job0_1"
 
+stub_routeActorJob_job0_1:
+li r6, 0
+cmpwi r5, 0
+bne exit_stub_routeActorJob_job0_1
+lwz r11, 0xE8(r3)
+lis r10, Player_vtable@ha
+addi r10, r10, Player_vtable@l
+cmpw r11, r10
+li r6, 1
+bne exit_stub_routeActorJob_job0_1
+li r6, 2
+
+exit_stub_routeActorJob_job0_1:
+mr r3, r6
+blr
+
+stub_routeActorJob_skipRight:
+li r3, 0
+cmpwi r5, 1
+bnelr
+li r3, 1
+blr
+
 hook_actor_job0_1:
 mflr r0
 stwu r1, -0x20(r1)
@@ -510,7 +529,17 @@ lis r4, strActor_job0_1@ha
 addi r4, r4, strActor_job0_1@l
 lis r5, currentEyeSide@ha
 lwz r5, currentEyeSide@l(r5)
-bl import.coreinit.hook_RouteActorJob
+lis r12, useStubHooks@ha
+lwz r12, useStubHooks@l(r12)
+cmpwi r12, 1
+lis r12, stub_routeActorJob_job0_1@ha
+addi r12, r12, stub_routeActorJob_job0_1@l
+beq routeActorJob_job0_1
+lis r12, import.coreinit.hook_RouteActorJob@ha
+addi r12, r12, import.coreinit.hook_RouteActorJob@l
+routeActorJob_job0_1:
+mtctr r12
+bctrl
 
 cmpwi r3, 0
 beq job0_1_normal
@@ -605,7 +634,17 @@ lis r4, strActor_job0_2@ha
 addi r4, r4, strActor_job0_2@l
 lis r5, currentEyeSide@ha
 lwz r5, currentEyeSide@l(r5)
-bl import.coreinit.hook_RouteActorJob
+lis r12, useStubHooks@ha
+lwz r12, useStubHooks@l(r12)
+cmpwi r12, 1
+lis r12, stub_routeActorJob_skipRight@ha
+addi r12, r12, stub_routeActorJob_skipRight@l
+beq routeActorJob_job0_2
+lis r12, import.coreinit.hook_RouteActorJob@ha
+addi r12, r12, import.coreinit.hook_RouteActorJob@l
+routeActorJob_job0_2:
+mtctr r12
+bctrl
 
 cmpwi r3, 1
 beq finish_hook_actor_job0_2
@@ -648,7 +687,17 @@ lis r4, strActor_job1_1@ha
 addi r4, r4, strActor_job1_1@l
 lis r5, currentEyeSide@ha
 lwz r5, currentEyeSide@l(r5)
-bl import.coreinit.hook_RouteActorJob
+lis r12, useStubHooks@ha
+lwz r12, useStubHooks@l(r12)
+cmpwi r12, 1
+lis r12, stub_routeActorJob_skipRight@ha
+addi r12, r12, stub_routeActorJob_skipRight@l
+beq routeActorJob_job1_1
+lis r12, import.coreinit.hook_RouteActorJob@ha
+addi r12, r12, import.coreinit.hook_RouteActorJob@l
+routeActorJob_job1_1:
+mtctr r12
+bctrl
 
 cmpwi r3, 1
 beq finish_hook_actor_job1_1
@@ -694,7 +743,17 @@ lis r4, strActor_job1_2@ha
 addi r4, r4, strActor_job1_2@l
 lis r5, currentEyeSide@ha
 lwz r5, currentEyeSide@l(r5)
-bl import.coreinit.hook_RouteActorJob
+lis r12, useStubHooks@ha
+lwz r12, useStubHooks@l(r12)
+cmpwi r12, 1
+lis r12, stub_routeActorJob_skipRight@ha
+addi r12, r12, stub_routeActorJob_skipRight@l
+beq routeActorJob_job1_2
+lis r12, import.coreinit.hook_RouteActorJob@ha
+addi r12, r12, import.coreinit.hook_RouteActorJob@l
+routeActorJob_job1_2:
+mtctr r12
+bctrl
 
 cmpwi r3, 1
 beq finish_hook_actor_job1_2
@@ -739,7 +798,17 @@ lis r4, strActor_job2_1@ha
 addi r4, r4, strActor_job2_1@l
 lis r5, currentEyeSide@ha
 lwz r5, currentEyeSide@l(r5)
-bl import.coreinit.hook_RouteActorJob
+lis r12, useStubHooks@ha
+lwz r12, useStubHooks@l(r12)
+cmpwi r12, 1
+lis r12, stub_routeActorJob_skipRight@ha
+addi r12, r12, stub_routeActorJob_skipRight@l
+beq routeActorJob_job2_1
+lis r12, import.coreinit.hook_RouteActorJob@ha
+addi r12, r12, import.coreinit.hook_RouteActorJob@l
+routeActorJob_job2_1:
+mtctr r12
+bctrl
 
 cmpwi r3, 1
 beq finish_hook_actor_job2_1
@@ -784,7 +853,17 @@ lis r4, strActor_job2_2@ha
 addi r4, r4, strActor_job2_2@l
 lis r5, currentEyeSide@ha
 lwz r5, currentEyeSide@l(r5)
-bl import.coreinit.hook_RouteActorJob
+lis r12, useStubHooks@ha
+lwz r12, useStubHooks@l(r12)
+cmpwi r12, 1
+lis r12, stub_routeActorJob_skipRight@ha
+addi r12, r12, stub_routeActorJob_skipRight@l
+beq routeActorJob_job2_2
+lis r12, import.coreinit.hook_RouteActorJob@ha
+addi r12, r12, import.coreinit.hook_RouteActorJob@l
+routeActorJob_job2_2:
+mtctr r12
+bctrl
 
 cmpwi r3, 1
 beq finish_hook_actor_job2_2
@@ -829,7 +908,17 @@ lis r4, strActor_job4@ha
 addi r4, r4, strActor_job4@l
 lis r5, currentEyeSide@ha
 lwz r5, currentEyeSide@l(r5)
-bl import.coreinit.hook_RouteActorJob
+lis r12, useStubHooks@ha
+lwz r12, useStubHooks@l(r12)
+cmpwi r12, 1
+lis r12, stub_routeActorJob_skipRight@ha
+addi r12, r12, stub_routeActorJob_skipRight@l
+beq routeActorJob_job4
+lis r12, import.coreinit.hook_RouteActorJob@ha
+addi r12, r12, import.coreinit.hook_RouteActorJob@l
+routeActorJob_job4:
+mtctr r12
+bctrl
 
 cmpwi r3, 1
 beq finish_hook_actor_job4

resources/BreathOfTheWild_BetterVR/patch_STUB_StereoRenderingDebugging.asm

@@ -8,50 +8,67 @@ useStubHooks:
 
 const_cameraHeightChange:
 .float 0.5
+.float -0.5
+; shouldn't use this, but we might upgrade to a bigger frame counter
+.float 0.5
+.float -0.5
 
+; r3 = layer->renderCamera
+; r11 = currentEyeSide
+; r12 = modifiedCopy_seadLookAtCamera
 stub_hook_GetRenderCamera:
-;blr
-lis r11, currentEyeSide@ha
-lwz r11, currentEyeSide@l(r11)
 cmpwi r11, 0
 beqlr ; don't modify right eye
 
+
+lis r12, currentFrameCounter@ha
+lwz r12, currentFrameCounter@l(r12)
+cmpwi r12, 0
+; camera.pos.y += 0.5 or -0.5 depending on the frame
+lis r11, const_cameraHeightChange@ha
+addi r11, r11, const_cameraHeightChange@l
+beq .+0x08
+addi r12, r12, 1
+lfs f12, 0(r11)
+
+
 ; camera->pos.y += 2.0; 
 lfs f12, 0x34+0x04(r3)
-lis r11, const_cameraHeightChange@ha
-lfs f13, const_cameraHeightChange@l(r11)
-;fadd f13, f13, f12
-stfs f13, 0x34+0x04(r3)
+fadds f12, f12, f13
+stfs f12, 0x34+0x04(r3)
 ; camera->at.y += 2.0;
 lfs f12, 0x40+0x04(r3)
-lis r11, const_cameraHeightChange@ha
-lfs f13, const_cameraHeightChange@l(r11)
-;fadd f13, f13, f12
-stfs f13, 0x40+0x04(r3)
+fadds f12, f12, f13
+stfs f12, 0x40+0x04(r3)
 ; camera->mtx.pos_y += 2.0;
 lfs f12, 0x1C(r3)
-lis r11, const_cameraHeightChange@ha
-lfs f13, const_cameraHeightChange@l(r11)
-;fadd f13, f13, f12
-stfs f13, 0x1C(r3)
+fadds f12, f12, f13
+stfs f12, 0x1C(r3)
 
 blr
 
 
 example_PerspectiveProjectionMatrix:
 .byte 1,1,0,0
-.int 0x3F9A678C
+; matrix
 .float 0.9300732,-107374176,-107374176,-107374176
 .float -107374176,0.86867154,-107374176,-107374176
 .float 0.06992682,-0.0352425,-1.000008,-1
 .float -107374176,-107374176,-0.2000008,0
+; deviceMatrix
 .float 0.9300732,-107374176,-107374176,-107374176
 .float -107374176,0.86867154,-107374176,-107374176
 .float 0.06992682,-0.0352425,-1.000008,-1
 .float -107374176,-107374176,-0.2000008,0
+; devicePosture
+.int 0
+; deviceZScale
 .float 1
+; deviceZOffset
 .float 0
+; lookAt vtable
 .int 0x1027B54C
+; near, far and fov
 .float 0.1
 .float 25000
 .float 1.7104228