arsenm accepted this revision. arsenm added a comment. This revision is now accepted and ready to land.
LGTM ================ Comment at: clang/test/CodeGen/isfpclass.c:2 +// RUN: %clang_cc1 -triple x86_64-linux-gnu -S -O1 -emit-llvm %s -o - | FileCheck %s + +inline __attribute__((always_inline)) _Bool func_isnan(float x) { ---------------- While already covered by existing tests, it might be nicer to test all the builtins in the same file with and without fenv access enabled Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D112932/new/ https://reviews.llvm.org/D112932 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits