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

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.