Seja bem vindo ao StackOverflow @Moises ;)
Na sua função você está apenas pegando o valor do input document.calcform.visor.value
, trantando ele novoValor = valor.replace(",",".")
e atribuindo ele a variavel valor
novamente, mas em momento nenhum está setando o valor do input com essa variavel.
Coloque isso no final de sua função e ja deve funcionar:
document.calcform.visor.value = valor;
Mas há um jeito mais simples de fazer o que você precisa:
function substituir(el) {
el.value = el.value.replace(",", ".");
}
<input type="text" onkeyup="substituir(this);" />