Differences

This shows you the differences between two versions of the page.

--- ms-20240504.cu [2024/06/20 08:47] – mino
+++ ms-20240504.cu [2024/10/16 15:05] (current) – mino
@@ Line 20: / Line 20: @@
   sys     0m0.272s
-== Note added on 2024.06.19 ==
+== Note added on 2024.06.20 ==
-The function void LastCols( ) in the code contains the following doubtful comparison:
+The function void LastCols( ) in the code contains the following questionable comparison:
       psingl = psingl
@@ Line 32: / Line 32: @@
                  or rev_max32_c == unsigned( v2 )
-because the upper 32 bits of v2 should be ignored.
+because the upper 32 bits of v2 should be ignored when compared with the 32bit unsigned rev_max32_c.
-However, this flaw is irrelevant to the validity of the result because this logical operation is totally unnecessary and has no effect for even N (including 6). And for N < 6, the upper 32 bits of v2 are always 0s and the cast has no effect in any way.
+However, this flaw is irrelevant to the validity of the result because this logical operation is totally unnecessary and has no effect for even N (including 6) because complements of any rows never appear as columns in even order magic squares. And for N < 6, the upper 32 bits of v2 are always 0s and the cast has no effect in any way.
-This part of the code and other parts related to 'complementary pair handling' should be removed from functions MakeCols() and LastCols() to optimize the performance for N=6. Complements of any rows never appear as columns in even order magic squares.
+This part of the code and other parts related to the self complemenarity check should be removed from functions MakeCols() and LastCols() in order to optimize the performance for N=6.
+We will gain 3% improvement of the performance by this optimization.
 <file cpp ms-20240504.cu>