This is an archive of the discontinued LLVM Phabricator instance.

[VE] Support VE architecture
ClosedPublic

Authored by kaz7 on Mar 17 2020, 5:31 PM.

Details

Summary

Modify LNT to let it know VE architecture in order to run
llvm-test-suite well on NEC SX-Aurora VE architecture.

Event Timeline

kaz7 created this revision.Mar 17 2020, 5:31 PM
kaz7 added a project: Restricted Project.Mar 17 2020, 5:33 PM
kaz7 added a comment.Mar 24 2020, 5:34 PM

Hi,

This is a patch to support NEC SX-Aurora TSUBASA Vector Engine (VE) architecture to compile llvm-test-suite.

Previously, I asked how to apply this patch at https://lists.llvm.org/pipermail/llvm-dev/2019-December/137537.html
and received suggestions to post it through phabricator.

So, I send this and add people actively work on lnt. Please let me know if you need more information.

Thanks in advance.

kristof.beyls accepted this revision.Mar 25 2020, 6:15 AM

This seems OK to me.
I'd just remark that these days, I think most people using LNT to compile/run/benchmark test-suite use "lnt runtest test-suite" rather than "lnt runtest nt".
Therefore, maybe it'd be better if you'd look into how to make that working on the VE architecture (i.e. look at file lnt/tests/test_suite.py).

This revision is now accepted and ready to land.Mar 25 2020, 6:15 AM
kaz7 added a comment.Mar 25 2020, 4:39 PM

Thank you about "lnt runtest test-suite" information. I tried it before, but had several errors not handled at that time. I definitely try it again later. At the moment, I appreciate landing this patch to nt.py. Thanks.

kaz7 added a comment.May 14 2020, 12:57 AM

Hi, this patch is accepted and landed like (https://github.com/llvm/llvm-lnt/commit/9c9f6f8aaca07b783b4a2db2b68f7878a61bd97a). Thank you.

However, this patch is not closed automatically. Should I close this one by myself? Other patches are automatically closed, so I am wondering. Thanks in advance.

kaz7 closed this revision.May 19 2020, 4:49 PM

Closing this one since it is landed already. Thank you for reviewing and landing this.