Algumas coisas para você verificar no código:
1º: A sua função calculaImc, depois do var imc = ..., você tem que colocar um return imc, para quando a função for chamada, ela retornar o valor do imc.
2º: Depois que você define o peso e a altura, e chama o calculaImc, você inverteu as ordens dos parâmetros, primeiro é a altura e depois o peso, e neste caso, você passou primeiro o peso e depois a altura. O correto: calculaImc(minhaAltura, meuPeso)
3º: Quando você chama a função, passando o peso e a altura, ok, você chamou, não pediu pra mostrar em lugar algum, e não salvou em nenhuma variável, portanto existem duas soluções - Ou você armazena essa chamada em uma variável, e passa essa variável la dentro do Math.round, ou então passa o calculaImc dentro do Math.round, mas passando os parâmetros, então ficaria assim - mostra(Math.round(calculaImc(minhaAltura, meuPeso));. (Eu prefiro com variável, fica menor e mais legível o código. Outro detalhe, não pode haver espaço entre o mostra e o parênteses, tem que estar colado). Se você optar por está ultima opção, em passar os parâmetros dentro do Math.round, então pode excluir a linha anterior, onde você chama o calculaImc.
Tenta ai, e qualquer dúvida comenta aqui.
