[PATCH] [mips] [IAS] Add support for LAReg with identical source and destination register operands.

Daniel Sanders daniel.sanders at imgtec.com
Thu Jun 18 04:15:02 PDT 2015


LGTM

> Daniel, is shortening IntermediateDstReg to TmpReg OK with you ?

>  You pointed that out in private, but since I ended up making the change, your position should be included in the public review.


I didn't mind either way. When we spoke, I was making a general comment that descriptive names are generally good but short, non-descriptive names aren't always bad. The obvious example is 'i', and 'j' for loop counters. Personally, I use TmpReg for trivial scratch registers and use more descriptive names when there are several of them (otherwise it starts to get confusing).


http://reviews.llvm.org/D9367

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list