[PATCH] D88888: [SystemZAsmParser] Treat VR128 separately in ParseDirectiveInsn().

[Ulrich Weigand via Phabricator via llvm-commits]

uweigand added a comment.

See inline comment.  Otherwise LGTM.



================
Comment at: llvm/lib/Target/SystemZ/SystemZRegisterInfo.td:127
                               (add (sequence "R%uD", 0, 15),
-                                   (sequence "F%uD", 0, 15),
-                                   (sequence "V%u", 0, 15)), 0/*allocatable*/>;
+                                   (sequence "F%uD", 0, 15)), 0/*allocatable*/>;
+
----------------
Given that we want to allow %v0..%v15 (just not %v16..%v31), I think the old definition actually was correct.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D88888/new/

https://reviews.llvm.org/D88888