[PATCH] D10640: [mips][microMIPS] Implement LWP and SWP instructions

Simon Dardis via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 19 07:42:32 PDT 2016


sdardis added a comment.

This patch looks mostly correct to me except it's missing a key issue, that for lwp rd and base cannot be the same. lwp is not guaranteed to be uninterruptible so an interrupt could render lwp not restartable if they are the same.

Fortunately we already have an similar example with jalr.hb. You need to extend MipsAsmParser::checkTargetMatchPredicate(MCInst &Inst) on line 3668 to account for this context sensitive predicate.


http://reviews.llvm.org/D10640





More information about the llvm-commits mailing list