cor3ntin added inline comments.
================ Comment at: clang/lib/AST/Interp/Function.h:180 + // FIXME: Is there a better way to get this information? + if (getName() == "__builtin_isnan") + return true; ---------------- tbaeder wrote: > cor3ntin wrote: > > You can compare getBuiltinID(), which is better than using strings, it's > > what we seem to be doing elsewhere > Yeah but that still sucks. Is there no way to query whether the builtin has > `t` set ("signature is meaningless, use custom typechecking"), regardless of > the ID or name? `AstContext.BuiltinInfo.hasCustomTypechecking(F->getBuiltinID()))` seems to do that but i have no clue if you could use it on all functions without running into issues Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D155369/new/ https://reviews.llvm.org/D155369 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits