[PATCH] D39089: [AArch64][SVE] Asm: Add SVE (Z) Register definitions and parsing support

Renato Golin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 6 09:26:09 PST 2017


rengolin accepted this revision.
rengolin added a comment.
This revision is now accepted and ready to land.

LGTM, without the extra whitespace change. Thanks!



================
Comment at: lib/Target/AArch64/AsmParser/AArch64AsmParser.cpp:1748
 
+};
 } // end anonymous namespace.
----------------
oops, still whitespace changes... :)

I always check the git diff locally before sending, as those pop up quite often.


https://reviews.llvm.org/D39089





More information about the llvm-commits mailing list