This is an archive of the discontinued LLVM Phabricator instance.

[Intrinsics] Re-remove experimental_vector_reduce intrinsics
ClosedPublic

Authored by dmgreen on Dec 1 2020, 10:57 AM.

Details

Summary

These appear to have been re-added by rGfbfb1c790982277eaa5134c2b6aa001e97fe828d, but probably should not have been? This removes the experimental versions of the reduction intrinsics again, leaving the non experimental versions.

Diff Detail

Event Timeline

dmgreen created this revision.Dec 1 2020, 10:57 AM
Herald added a project: Restricted Project. · View Herald TranscriptDec 1 2020, 10:57 AM
dmgreen requested review of this revision.Dec 1 2020, 10:57 AM
nikic accepted this revision.Dec 1 2020, 11:20 AM

LGTM

This revision is now accepted and ready to land.Dec 1 2020, 11:20 AM
sstefan1 accepted this revision.Dec 1 2020, 12:50 PM

Definitely not intentional, sorry.

Thanks. It looked like a merge conflict gone awry, but I wanted to be sure it wasn't done purposefully.