This is an archive of the discontinued LLVM Phabricator instance.

[VE,#4] Target vector intrinsics
AbandonedPublic

Authored by simoll on Dec 11 2019, 3:13 AM.

Details

Reviewers
jdoerfert
mclow.lists
Group Reviewers
Restricted Project
Restricted Project
Restricted Project
Summary

This is patch #4 in the patch series for the VE backend for NEC-SX Aurora.
This patch builds on the runtime library patch of https://reviews.llvm.org/D71340

This patch implements vel intrinsics for SX-Aurora. These target-specific vector intrinsics are documented here.

Event Timeline

simoll created this revision.Dec 11 2019, 3:13 AM
Herald added projects: Restricted Project, Restricted Project, Restricted Project, Restricted Project, Restricted Project. · View Herald Transcript

As a general rule, I think it would be preferable for patches such as this one to be split up further, especially when they touch common code. For example, why does a patch by the name "Target vector intrinsics" contain a change to SjLjEHPrepare?

As a general rule, I think it would be preferable for patches such as this one to be split up further, especially when they touch common code. For example, why does a patch by the name "Target vector intrinsics" contain a change to SjLjEHPrepare?

This patch is not intended to be committed as-is but gives an overview of the required changes. It's the a patch of a series for the VE backend and includes all the earlier patches.

simoll added a project: Restricted Project.Dec 11 2019, 5:48 AM
simoll abandoned this revision.Nov 17 2020, 10:56 AM

A good share of VE vector intrinsics have been upstreamed (see test/CodeGen/VE/VELIntrinsics/). No need to keep this reference patch around any longer.

Herald added a reviewer: Restricted Project. · View Herald TranscriptNov 17 2020, 10:56 AM
Herald added projects: Restricted Project, Restricted Project. · View Herald Transcript
Herald added a reviewer: Restricted Project. · View Herald Transcript
Herald added a reviewer: Restricted Project. · View Herald Transcript
Herald added subscribers: dang, sstefan1. · View Herald Transcript