[PATCH] D92185: [SystemZ] Adding extra extended mnemonics for SystemZ target

Anirudh Prasad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 27 07:59:12 PST 2020


anirudhp added a comment.

> This looks good to me.   The only thing I'm wondering is whether we shouldn't complete the NOP set by adding a 6-byte "jgnop".   (I guess this is called jlnop with HLASM, but in order to keep with the jg* naming scheme in GAS we probably ought to use jgnop here.)

This is a good idea. I have updated the patch to reflect the same. Thanks for the hint!


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

https://reviews.llvm.org/D92185



More information about the llvm-commits mailing list