[PATCH] [PPC64] Add 64-bit Vector Integer Arithmetic Instructions

hfinkel at anl.gov hfinkel at anl.gov
Tue Mar 3 19:36:04 PST 2015


================
Comment at: lib/Target/PowerPC/PPCSchedule.td:376
@@ -375,3 +375,3 @@
 //    tlbie      IIC_LdStDCBF
 //    tlbsync    IIC_SprTLBSYNC
 //    tw         IIC_IntTrapW
----------------
wschmidt wrote:
> Question for Hal:  Why do we have this table of comments that needs to be updated as new instructions are added?  Not sure whether it provides value, and it's an extra bit of noise in the process.  Just curious.  Others have noted that this list is not currently complete in any case.  I'd personally rather just see it go away.
I don't know; this table pre-dates my involvement with LLVM. I agree, however, that the table is incomplete, and likely adds little value. Feel free to remove it.

http://reviews.llvm.org/D7959

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






More information about the llvm-commits mailing list