Something I forgot when I added the Cortex-R82.
Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Can you add a test line to the fuse-aes test? The postra feature is harder to test very well.
Thanks Dave, I got so excited about not having to test post RA scheduling I forgot about aes ;-)
llvm/lib/Target/AArch64/AArch64.td | ||
---|---|---|
677 | If you are adding crypto (which doesn't sound like a bad idea to me), should there be some other tests for that? If you want to keep this simple and do that in a separate patch, you could just add -mattr=+crypto to the r82 run line. |
llvm/lib/Target/AArch64/AArch64.td | ||
---|---|---|
677 | I discovered I need crypto otherwise we get "can't match aes intrinsics" for the fuse aes tests. |
If you are adding crypto (which doesn't sound like a bad idea to me), should there be some other tests for that?
If you want to keep this simple and do that in a separate patch, you could just add -mattr=+crypto to the r82 run line.