[PATCH] D69849: [mips] Implement Octeon+ `saa` and `saad` instructions
Simon Atanasyan via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Nov 7 00:13:49 PST 2019
atanasyan added a comment.
Thanks for review.
In D69849#1735518 <https://reviews.llvm.org/D69849#1735518>, @Petar.Avramovic wrote:
> Correction from previous comment, tests for SaaAddr and SaadAddr are already present.
Anyway I decided to add more tests. In particular, check saa <reg>, <symbol> expansion in case of PIC.
> Remaining question is about flags for tests,
> for gnu-as both `-mips32` and `-mips64` are not compatible with `-march=octeon+`, minimum requirement is `-mips64r2`, but feature compatibility could be addressed in another patch.
You are right. Now it's possible to pass `-mips64 -march=octeon` to the `clang` and do not get any errors. I'm going to solve that as a separate patch for the driver.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D69849/new/
https://reviews.llvm.org/D69849
More information about the llvm-commits
mailing list