Skip to content

Commit ba6ddf8

Browse files
committed
the guard around CheckRel/CheckBwd/CheckUlps was unneecssary
1 parent a34c563 commit ba6ddf8

1 file changed

Lines changed: 0 additions & 7 deletions

File tree

ops/dot_test.cc

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -734,17 +734,10 @@ class DotStats {
734734
void Check() const {
735735
CheckMuls();
736736
CheckL1();
737-
#if !HWY_ARCH_ARM_A64
738-
// CheckRel/CheckBwd/CheckUlps thresholds are tuned for x86; on aarch64
739-
// the compensated dot product has slightly higher relative error
740-
// (see the explicit "Extremely high error on aarch64" comments below for
741-
// precedent). Skip them on aarch64 rather than maintain two sets of
742-
// platform-specific bounds.
743737
CheckRel();
744738
CheckBwd();
745739
// No need to check bits, it is a monotonic function of rel.
746740
CheckUlps();
747-
#endif
748741

749742
// We do not check times because they can be noisy/nonportable, but
750743
// `kAddTwoProd` is only about 10% slower than `kKahan`, and about 1.5 times

0 commit comments

Comments
 (0)