Skip to content

Commit

Permalink
fix comments and remove not needed ensure_started
Browse files Browse the repository at this point in the history
  • Loading branch information
albestro committed Jul 20, 2023
1 parent 35654f8 commit 3d21dc9
Showing 1 changed file with 10 additions and 11 deletions.
21 changes: 10 additions & 11 deletions include/dlaf/eigensolver/tridiag_solver/merge.h
Original file line number Diff line number Diff line change
Expand Up @@ -743,7 +743,7 @@ void mergeSubproblems(const SizeType i_begin, const SizeType i_split, const Size
// Step #2
//
// i2 (in) : initial <--- pre_sorted
// i5 (out) : initial <--- sorted by coltype
// i5 (out) : initial <--- sorted by coltype
// i3 (out) : initial <--- deflated
// i4 (out) : deflated <--- sorted by coltype
//
Expand All @@ -756,14 +756,13 @@ void mergeSubproblems(const SizeType i_begin, const SizeType i_split, const Size
// by coltype)
// - set deflated diagonal entries of `U` to 1 (temporary solution until optimized GEMM is implemented)
//
// | U | U | D | D | | | DF | DF | U: UpperHalf
// | U | U | D | D | | | DF | DF | D: Dense
// | | | D | D | L | L | DF | DF | L: LowerHalf
// | U | U | D | D | | | DF | DF | U: UpperHalf
// | U | U | D | D | | | DF | DF | D: Dense
// | | | D | D | L | L | DF | DF | L: LowerHalf
// | | | D | D | L | L | DF | DF | DF: Deflated
// | | | D | D | L | L | DF | DF |
//
auto k = ex::ensure_started(stablePartitionIndexForDeflation(i_begin, i_end, ws_h.c, ws_hm.i2, ws_h.i3,
ws_hm.i5)) |
auto k = stablePartitionIndexForDeflation(i_begin, i_end, ws_h.c, ws_hm.i2, ws_h.i3, ws_hm.i5) |
ex::split();

copy(idx_begin_tiles_vec, sz_tiles_vec, ws_hm.i5, ws.i5);
Expand All @@ -774,15 +773,15 @@ void mergeSubproblems(const SizeType i_begin, const SizeType i_split, const Size
copy(idx_begin_tiles_vec, sz_tiles_vec, ws_hm.d1, ws_h.d0);

//
// i3 (in) : initial <--- deflated
// i2 (out) : initial ---> deflated
// i3 (in) : initial <--- deflated
// i2 (out) : deflated <--- initial
//
invertIndex(i_begin, i_end, ws_h.i3, ws_hm.i2);

//
// i5 (in) : sort by coltype ---> initial
// i2 (in) : initial ---> deflated
// i4 (out) : sort by col type ---> deflated
// i5 (in) : initial <--- sort by coltype
// i2 (in) : deflated <--- initial
// i4 (out) : deflated <--- sort by col type
//
// This allows to work in rank1 solver with columns sorted by type, so that they are well-shaped for
// an optimized gemm, but still keeping track of where the actual position sorted by eigenvalues is.
Expand Down

0 comments on commit 3d21dc9

Please sign in to comment.