[PATCH] D21809: [SystemZ] Add support for .insn and .word/.short/.long/.quad directives

Marcin Koƛcielnicki via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 29 02:25:12 PDT 2016


koriakin requested changes to this revision.
koriakin added a reviewer: koriakin.
koriakin added a comment.
This revision now requires changes to proceed.

See inline comments.  Also, this might not be the good level to have .insn handling - for one, we forfeit any hope of handling fixups in immediate operands (which is necessary to handle pc-relative forms correctly). It might be better to represent these directives as special machine instructions, one per form.


http://reviews.llvm.org/D21809





More information about the llvm-commits mailing list