This is an archive of the discontinued LLVM Phabricator instance.

[X86] Update avx512vbmi2vl intrinsic tests to avoid adds
ClosedPublic

Authored by LuoYuanke on Mar 13 2022, 6:18 PM.

Details

Summary

As noticed in D119654, by adding the masked intrinsics results together
we can end up with the selects being canonicalized away from the
intrinsic - this isn't what we want to test here so replace with a
insertvalue chain into a aggregate instead to retain all the results.

Diff Detail

Event Timeline

LuoYuanke created this revision.Mar 13 2022, 6:18 PM
Herald added a project: Restricted Project. · View Herald TranscriptMar 13 2022, 6:18 PM
Herald added a subscriber: pengfei. · View Herald Transcript
LuoYuanke requested review of this revision.Mar 13 2022, 6:18 PM
Herald added a project: Restricted Project. · View Herald TranscriptMar 13 2022, 6:18 PM
This revision was not accepted when it landed; it landed in state Needs Review.Mar 13 2022, 6:30 PM
This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.