This is an archive of the discontinued LLVM Phabricator instance.

[ValueTracking] Fix computeKnownFPClass with canonicalize
ClosedPublic

Authored by piotr on May 15 2023, 5:22 AM.

Details

Summary

Update code that assumes llvm.canonicalize only handles scalars,
by adding a call to getScalarType().
This is fine, as the intrinsic is trivially vectorizable.

Introduced in D147870, and uncovered by D148065.

Diff Detail

Unit TestsFailed

Event Timeline

piotr created this revision.May 15 2023, 5:22 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 15 2023, 5:22 AM
Herald added a subscriber: hiraditya. · View Herald Transcript
piotr requested review of this revision.May 15 2023, 5:22 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 15 2023, 5:22 AM
foad accepted this revision.May 15 2023, 5:34 AM
This revision is now accepted and ready to land.May 15 2023, 5:34 AM
This revision was landed with ongoing or failed builds.May 15 2023, 5:51 AM
This revision was automatically updated to reflect the committed changes.