Agreed.

The TODO line should be removed.  I actually changed it back to faulting the VM on error.  The TODO was from a previous revision where I just let it continue.

If any of the instruction execution functions return false, the VM faults.

I'll simplify this code and remove the TODO.