[PATCH] D75428: [MC][ARM] add implicit immediate form for ldrsbt/ldrht/ldrsht

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 2 12:55:25 PST 2020


efriedma added a comment.

Generally, we prefer to use an alias unless there's some reason we need logic written in C++.  For example, some "instructions" don't translate to a single instruction.  And some instructions have complex logic for which encoding to use.

In this case, ARMAsmPseudo seems like overkill; an InstAlias should be sufficient.


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

https://reviews.llvm.org/D75428





More information about the llvm-commits mailing list