Как использовать одну функцию для нескольких входов в javascript

e.target относится к <input>, над которым вы работаете, поэтому просто измените

document.getElementById(...).value = val;

к

e.target.value = val;

и вы сможете использовать одну и ту же функцию для обоих входов. Также, вероятно, больше не нужны ids.

<input type = "text" name = "samount" class = "form-control"  @keyup=${this.formatCurrency} >
<input type = "text" name = "tamount" class = "form-control" @keyup=${this.formatCurrency} >

а также

formatRCurrency(e){
  var value = e.target.value;
  value = value.replace(/[^0-9\.]/g,'');      
  if (value !== "") {
    var valArr = val.split('.');
    valArr[0] = (parseInt(valArr[0],10)).toLocaleString();
    val = valArr.join('.');
  }      
  e.target.value = val;
}

(обратите внимание, что ваша переменная myinput на самом деле содержит нить, а не HTMLInputElement, что может сбивать с толку — лучше использовать точные имена переменных)

Простой способ добиться этого — внимательно изучить атрибут id целевого элемента и принять соответствующие меры. Например...

function eventHandler(e) {
    var targetID = e.target.id;

    switch(targetID) {
        case "samount":
            /* 'samount' specific process */
            break;
        default:
           /* 'tamount' specific process */
    }
}

Если вы хотите выполнить один и тот же процесс независимо от активного элемента, свойство event.target содержит ссылку на этот элемент...

function eventHandler(e) {
    var element = e.target;
    var val = element.value;
    // process 'val' etc ....
    element.value = new_value;
}

Надеюсь, это поможет. :)

• переключатель @ MDN
• event.target @ MDN