Além do problema em usar o operador `||` (or) em vez do `&&` (and), onde o `&&` significa que ambas as condições devem ser *true*, neste caso seria melhor usar um `if else` e um `else` (caso não queira usar `Math.max()` e manter a estrutura de `if`'s), pois não é necessário passar por cada `if` se você quer que apenas 1 conjunto de condições seja satisfeita.

Por exemplo, no primeiro `if` você já "mata" a questão verificando se o `valor1` é maior que os outros dois valores:

    if(valor1 >= valor2 && valor1 >= valor3){
      alert("O valor maior é "+valor1)
    }else if(valor2 >= valor3){
      alert("O valor maior é "+valor2)
    }else{
      alert("O valor maior é "+valor3)
    }

Se `valor1` for maior ou igual a `valor2` e `valor3` a verificação irá ignorar os dois outros blocos (`else if` e `else`) já encerrando a verificação nas primeiras condições.

Se `valor1` for menor que `valor2` e maior ou igual a `valor3`, não satisfará o primeiro `if` passando pro `else if`, logo, já é descartado que `valor1` é o maior número, então, se `valor2` for maior ou igual a `valor3`, a verificação acaba aí, sendo `valor2` o maior  número se ele for maior que `valor3` ou então, se for igual a `valor3`, ambos são os valores maiores. Agora, se `valor2` for menor que `valor3`, então irá cair no `else` porque nenhum dos dois `if`'s foram atendidos, e com isso `valor3` é o maior número **ou** `valor1` e `valor3` são iguais e são os maiores números.

**Testes de mesa:**

    valor1 = 1
    valor2 = 2
    valor3 = 3
    
         if: condição 1 = falso, porque valor1 não é igual nem maior que valor2
                          (nem precisa verificar a segunda condição)
    else if: falso, porque 2 não é maior nem igual a valor3, logo, obrigatoriamente
             irá cair no else

    > resultado: número maior = 3

    ------------------------------

    valor1 = 3
    valor2 = 2
    valor3 = 1
    
         if: condição 1 = verdadeiro, porque valor1 é maior ou igual que valor2
             condição 2 = verdadeiro, porque valor1 é maior ou igual que valor3
    else if: ignorado
       else: ignorado

    > resultado: número maior = 3

    ------------------------------

    valor1 = 1
    valor2 = 2
    valor3 = 1
    
         if: condição 1 = falso, porque valor1 é menor que valor2
                          (nem precisa verificar a segunda condição)
    else if: verdadeiro, porque valor2 é maior que valor3
       else: ignorado

    > resultado: número maior = 2

    ------------------------------

    valor1 = 1
    valor2 = 1
    valor3 = 1
    
         if: condição 1 = verdadeiro, porque valor1 é igual a valor2
             condição 2 = verdadeiro, porque valor1 é igual a valor3
    else if: ignorado
       else: ignorado

    > resultado: número maior = 1

    ------------------------------

    valor1 = 1
    valor2 = 2
    valor3 = 2
    
         if: condição 1 = falso, porque valor1 é menor que valor2
             (nem precisa verificar a segunda condição)
    else if: verdadeiro, porque valor2 é igual a valor3
       else: ignorado

    > resultado: número maior = 2

**Teste:**

<!-- begin snippet: js hide: false console: true babel: false -->

<!-- language: lang-js -->

    var valor1 =Number(prompt("Digite hum numero: "))
    var valor2 =Number(prompt("Digite hum numero: "))
    var valor3 =Number(prompt("Digite hum numero: "))

    if(valor1 >= valor2 && valor1 >= valor3){
      alert("O valor maior é "+valor1)
    }else if(valor2 >= valor3){
      alert("O valor maior é "+valor2)
    }else{
      alert("O valor maior é "+valor3)
    }

<!-- end snippet -->