On 27/10/15 11:18, James Greenhalgh wrote:

  ;; -------------------------------------------------------------------
@@ -932,6 +934,8 @@
                                 UNSPEC_SQSHRN UNSPEC_UQSHRN
                                 UNSPEC_SQRSHRN UNSPEC_UQRSHRN])

+(define_int_iterator SQRDMLAH [UNSPEC_SQRDMLAH UNSPEC_SQRDMLSH])
+

This name does not make it clear that you will iterate over an "A" and an
"S" form. I'd like to see a clearer naming choice, RDMAS? SQRDMLHADDSUB? etc.

SQRDMLHADDSUB is a little difficult to read. How about SQRDMLH_AS, to keep the link to the instruction?

Matthew


Reply via email to