native: cleanup implementation comments about shifts
MODIFY
native/dlaqr04.go
native/dlaqr04.go
@@ -419,9 +419,9 @@ func (impl Implementation) Dlaqr04(wantt, wantz bool, n, ilo, ihi int, h []float
// Shuffle shifts into pairs of real shifts and pairs of
// complex conjugate shifts using the fact that complex
// conjugate shifts are already adjacent to one another.
- // TODO(vladimir-ch): Dlaqr5 also does this shuffling.
- // It seems that the one in Dlaqr5 is redundant but
- // check.
+ // TODO(vladimir-ch): The shuffling here could probably
+ // be removed but I'm not sure right now and it's safer
+ // to leave it.
for i := kbot; i > ks+1; i -= 2 {
if wi[i] == -wi[i-1] {
continue

MODIFY
native/dlaqr5.go
native/dlaqr5.go
@@ -124,9 +124,8 @@ func (impl Implementation) Dlaqr5(wantt, wantz bool, kacc22 int, n, ktop, kbot,
}
// Note: lapack says that nshfts must be even but allows it to be odd
- // anyway. At the moment, we panic above if nshfts is not even, so
- // reducing it by one is unnecessary (it seems that Dlaqr0 and Dlaqr4
- // indeed use only even nshfts).
+ // anyway. We panic above if nshfts is not even, so reducing it by one
+ // is unnecessary. The only caller Dlaqr04 uses only even nshfts.
//
// The original comment and code from lapack-3.6.0/SRC/dlaqr5.f:341:
// * ==== NSHFTS is supposed to be even, but if it is odd,

native: cleanup implementation comments about shifts