Changeset e0330d2c for tests/array-collections

Timestamp:

Dec 18, 2024, 6:37:11 PM (10 months ago)

Author:

Michael Brooks <mlbrooks@…>

Branches:

master

Children:

e0f3bd2

Parents:

faf9e19

Message:

Remove unnecessary special case for zero_t in code gen that caused unused-variable warning in thunks.

Add instructions for tidying warnings (with the present zero_t case as its running example) as a "proposal."

The code gen change is safe because a corresponding case for one_t was always "missing" and the the variable at issue is never used meaningfully. (Prior work has size_t lowered as int. While that's a dubious choice, it's beyond what I'm fixing here.)

Mark these tests as now clean of warnings:

array-collections/array-basic
ctrl-flow/loopctrl
vector
vector_math/vec2_int
vector_math/vec2_float
vector_math/vec3_float
vector_math/vec4_float
zero_one

File:

• tests/array-collections/array-basic.cfa (modified) (1 diff)

tests/array-collections/array-basic.cfa

@@ -12 +12 @@
     // numeric subscripts
     ptrdiff_t ix = 1, iy = 2, iz = 3;
-    array( float, Ny, Nz ) & yz = xyz[ix];
-    array( float, Nz ) & z = xyz[ix][iy];
-    float & val = xyz[ix][iy][iz];
+    array( float, Ny, Nz ) & yz = xyz[ix];      (void) yz;
+    array( float, Nz ) & z = xyz[ix][iy];       (void) z;
+    float & val = xyz[ix][iy][iz];              (void) val;
 
     // deferral subscript