[PATCH] D70758: [PowerPC][NFC] Rename record instructions to use r suffix instead of o
    Justin Hibbits via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Mon Dec  2 13:58:00 PST 2019
    
    
  
jhibbits requested changes to this revision.
jhibbits added a comment.
I second @nemanjai, the lowercase 'o' suffix stands out to me as a clear indicator of it being punctuation, not a letter in the instruction.  I would much rather see ANDIo than ANDIr, the 'r' just doesn't look right when reading over it.  My vote is to reject.  It looks like a lot of churn without much benefit.
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D70758/new/
https://reviews.llvm.org/D70758
    
    
More information about the llvm-commits
mailing list