[PATCH] D45204: [X86][MIPS][ARM] New machine instruction property 'isMoveReg'

Nikola Prica via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 13 06:26:24 PDT 2018


NikolaPrica added a comment.

> Why can't you use that hook then?  Make it available for all targets in TargetInstrInfo.  I think that adding such bits to the instruction descriptor is a bad idea in general.

I've firstly implemented like that but I've got comment that it might be better to implement it through table description instead of using multiple switch-case constructions for every register move instruction.


https://reviews.llvm.org/D45204





More information about the llvm-commits mailing list