This is an archive of the discontinued LLVM Phabricator instance.

Verifier: Enforce value of llvm.is.fpclass test mask
ClosedPublic

Authored by arsenm on Dec 9 2022, 6:20 PM.

Details

Summary

As requested in D137811

Diff Detail

Event Timeline

arsenm created this revision.Dec 9 2022, 6:20 PM
Herald added a project: Restricted Project. · View Herald TranscriptDec 9 2022, 6:20 PM
Herald added a subscriber: hiraditya. · View Herald Transcript
arsenm requested review of this revision.Dec 9 2022, 6:20 PM
Herald added a project: Restricted Project. · View Herald TranscriptDec 9 2022, 6:20 PM
Herald added a subscriber: wdng. · View Herald Transcript
jyknight accepted this revision.Dec 10 2022, 5:43 PM
jyknight added inline comments.
llvm/lib/IR/Verifier.cpp
5016

I might avoid saying "has 10 bits" in the error string since that seems likely to forget to be updated. Maybe something like "Unsupported bits passed to llvm.is.fpclass test mask."? (Just choose something and commit.)

This revision is now accepted and ready to land.Dec 10 2022, 5:43 PM