zahiraam added inline comments.
================ Comment at: clang/test/CodeGen/arithmetic-fence-builtin.c:73 int subit(float a, float b, float *fp) { - // CHECKFAST: define {{.*}}@subit(float noundef %a, float noundef %b{{.*}} + // CHECKPRECISE: define {{.*}}@subit(float noundef %a, float noundef %b{{.*}} *fp = __arithmetic_fence(a - b); ---------------- bader wrote: > Why is this check removed for SPIR target? Added. ================ Comment at: clang/test/CodeGen/arithmetic-fence-builtin.c:74 int subit(float a, float b, float *fp) { - // CHECKFAST: define {{.*}}@subit(float noundef %a, float noundef %b{{.*}} + // CHECKPRECISE: define {{.*}}@subit(float noundef %a, float noundef %b{{.*}} *fp = __arithmetic_fence(a - b); ---------------- bader wrote: > What is different for SPIR target here? The RUN at line 16 with -DFAST would generate a warning: "#pragma float_control is not supported on this target". That's why I separated that check from the others. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D142583/new/ https://reviews.llvm.org/D142583 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits