[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