Thread

  1. [PATCH] Add pg_lfind8_nonzero()

    cca5507 <cca5507@qq.com> — 2025-12-14T13:33:00Z

    Hi hackers,
    
    I'd like to add pg_lfind8_nonzero() to optimize some code like this:
    
    ```
    	for (i = 0; i < numberOfAttributes; i++)
    	{
    		if (isnull[i])
    		{
    			hasnull = true;
    			break;
    		}
    	}
    ```
    
    With pg_lfind8_nonzero(), we can write the code like this:
    
    ```
    	if (likely(numberOfAttributes > 0))
    		hasnull = pg_lfind8_nonzero((uint8 *) isnull, numberOfAttributes);
    ```
    
    The pg_lfind8_nonzero() is faster because we can handle 8 bool values at a time.
    
    v1-0001 add the pg_lfind8_nonzero().
    
    v1-0002 use the pg_lfind8_nonzero() in some places.
    
    v1-0003 add a extension "test_patch" only for testing, the test like this:
    
    ```
        for (i = 0; i < 1000; i++)
        {
            for (int j = 0; j < natts; j++)
            {
                if (isnull[j])
                {
                    hasnull = true;
                    break;
                }
            }
        }
    =======================
        for (i = 0; i < 1000; i++)
        {
            if (likely(natts > 0))
                hasnull = pg_lfind8_nonzero((uint8 *) isnull, natts);
        }
    ```
    
    create extension test_patch;
    # 4 is the natts
    select test_head(4);
    select test_patch(4);
    
    Test result:
    natts: 4   head: 1984ns  patch: 2094ns
    natts: 8   head: 3196ns  patch: 641ns
    natts: 16  head: 4589ns  patch: 752ns
    natts: 32  head: 8036ns  patch: 1152ns
    natts: 64  head: 19367ns patch: 2455ns
    natts: 128 head: 33445ns patch: 4018ns
    
    Looking forward to your comments!
    
    --
    Regards,
    ChangAo Chen