andykaylor wrote:

> LGTM. Thanks. I had to double check, but it looks like the LIT test 
> `ffp-contract-fast-honor-pragma-option.cpp` has the same behavior with and 
> without this patch. Right? Aren't there already tests that check the output 
> of the option?

Yes, the `ffp-contract-fast-honor-pragma-option.cpp` is testing existing 
behavior. The existing tests didn't cover this.

https://github.com/llvm/llvm-project/pull/104857
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to