[Executorch][Portable] Dont upcast to double for sigmoid
Pull Request resolved: https://github.com/pytorch/executorch/pull/6892 Upcasting to double for compute precision may not be aten compliant. Reason for internal test change: Apparently running on broadwell CPU vs test runner with Cooper lake gives different results for this change. Without this change: Both broadwell and Cooper lake will produce "Once upon a time, there was a little" With this change: Broadwell still produces "Once upon a time, there was a little", while Cooperlake produces "Once upon a time, there was a girl". So one possibility is that that some XNNPACK kernel for Cooper lake is produces slightly different numerical result that propagates through. Still landing this change since upcasting to double for compute, does not seem necessary. ghstack-source-id: 253832495 @exported-using-ghexport Differential Revision: [D65928920](https://our.internmc.facebook.com/intern/diff/D65928920/) Co-authored-by: Kimish Patel <kimishpatel@fb.com>
H
Hansong committed
c242a59e5860fd1761fef7c379c69a06b6bfa910
Parent: 3b475e3
Committed by GitHub <noreply@github.com>
on 11/18/2024, 6:07:46 PM