fix: avoid duplicated definition

[yongweiy]

gather.c turns

import nki.isa.constants as constants
from nki.isa.constants import oob_mode

def dma_copy(
    dst,
    src,
    dst_rmw_op: np.ufunc = None,
    oob_mode=constants.oob_mode.error,
    dge_mode=dge_mode.unknown,
    name=None,
) -> None: ...

into

nki.isa.neuron_isa.dma_copy(dst,src,dst_rmw_op = None,oob_mode = nki.isa.constants.oob_mode.error,dge_mode = nki.isa.constants.dge_mode.unknown,name = None): ...

nki.isa.dma_copy(dst,src,dst_rmw_op = None,oob_mode = nki.isa.constants.oob_mode.error,dge_mode = dge_mode.unknown,name = None): ...

Default value dge_mode.unknown cannot be resolved later by KLR when the second definition is used.

[yongweiy]

With the update, the globals have the alias:

nki.isa.dma_copy = nki.isa.neuron_isa.dma_copy

[govereau]

Thanks!

[govereau]

This should be:

def->name = lean_string_cstr(def->str);

name and str should refer to the same Lean object.
I think it may not matter, but this is what the comments say is true about this structure.