Review Board 2.0.15


ARM: Fix type comparison warnings in Neon

Review Request #220 - Created Aug. 23, 2010 and submitted

Information
Ali Saidi
gem5
Reviewers
Default
ARM: Fix type comparison warnings in Neon.

   
Review request changed
Updated (Aug. 23, 2010, 2:38 a.m.)

Summary:

-# HG User Gabe Black <gblack@eecs.umich.edu>
+ARM: Fix type comparison warnings in Neon
Ship it!
Posted (Aug. 23, 2010, 5:06 a.m.)
I asume that you have a meaningful commit message and that reviewboard screwed up.  I don't want to be too anal here so feel free to overrule me, but is ltz a standard name in ARM?  If not, perhaps we should try to standardize these names somehow.  In Python, comparisons are named lt, gt, geq, leq, eq, and neq.  It's not a big deal, but I didn't immediately know what ltz meant without looking at the code.  Anyway I leave it up to you.
  1. ltz -- less that zero. I think that is pretty standard and since it's used a fair amount short is nice. I'll add a comment to neon.isa.