[PATCH] D92156: [PowerPC] Add support for "tlbiel" with two arguments

Jinsong Ji via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 22 16:16:31 PST 2020


jsji added a comment.

Yes, Clang still reject 5 operands, because that is what this patch should do. 😀

OK, so you confirmed that usage of `tlbiel` is for old arch with old format.

So we should update this patch to:

1. Support 5 operands `tlbie`
2. Add 2 operands InstAlias for `tlbie`
3. Support 5 operands `tlbiel`
4. Add 1 operand InstAlias for `tlbiel`
5. Add `tlbiel Rs, L` support for old arch.

I am OK with splitting 1/2 , and 3/4/5 into two patches.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D92156



More information about the llvm-commits mailing list