This is an archive of the discontinued LLVM Phabricator instance.

[clang][Interp] Implement __builtin_isfpclass
ClosedPublic

Authored by tbaeder on Jul 15 2023, 11:59 PM.

Diff Detail

Event Timeline

tbaeder created this revision.Jul 15 2023, 11:59 PM
Herald added a project: Restricted Project. · View Herald TranscriptJul 15 2023, 11:59 PM
tbaeder requested review of this revision.Jul 15 2023, 11:59 PM
Herald added a project: Restricted Project. · View Herald TranscriptJul 15 2023, 11:59 PM
Herald added a subscriber: cfe-commits. · View Herald Transcript
aaron.ballman added inline comments.Jul 31 2023, 5:53 AM
clang/lib/AST/Interp/InterpBuiltin.cpp
225

Should we add an assert that [1] is a valid index to read from?

232

Shouldn't this be pushing an int? (e.g., might not be 32 bits) -- same for the other builtins, now that I look at them.

clang/lib/AST/Interp/PrimType.h
135

BitInt comes to mind. ;-)

tbaeder marked an inline comment as done.Jul 31 2023, 8:10 AM
tbaeder added inline comments.
clang/lib/AST/Interp/InterpBuiltin.cpp
232

Yes, this is also covered by https://reviews.llvm.org/D155568

aaron.ballman accepted this revision.Jul 31 2023, 10:46 AM

LGTM

clang/lib/AST/Interp/InterpBuiltin.cpp
232

Ah, bitten by the massive patch stack again. :-)

This revision is now accepted and ready to land.Jul 31 2023, 10:46 AM
tbaeder closed this revision.Aug 1 2023, 9:15 AM
tbaeder marked an inline comment as done.

Also forgot the link in the commit message here. This is closed by https://github.com/llvm/llvm-project/commit/6ba4b213346fad36634a5571feda4f4481097c3a