[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