Skip to content

Conversation

@fsx950223
Copy link
Contributor

@fsx950223 fsx950223 commented Jul 11, 2025

The PR may fix pa compile bug #637 . Can't verify since lack of details.

@tjtanaa
Copy link
Contributor

tjtanaa commented Jul 11, 2025

Will it be possible to also address the compilation warning message in this PR:

hipcc -fPIC -DUSE_ROCM -DENABLE_FP8 -O3 -std=c++17 -DLEGACY_HIPBLAS_DIRECT -DUSE_PROF_API=1 -D_HIP_PLATFORM_HCC=1 -DHIP_PLATFORM_AMD=1 -UHIP_NO_HALF_CONVERSIONS_ -U_HIP_NO_HALF_OPERATORS_ -mllvm --amdgpu-kernarg-preload-count=16 -Wno-unused-result -Wno-switch-bool -Wno-vla-cxx-
extension -Wno-undefined-func-template -fgpu-flush-denormals-to-zero -fno-offload-uniform-block -mllvm -enable-post-misched=0 -mllvm -amdgpu-early-inline-all=true -mllvm -amdgpu-function-calls=false --offload-arch=gfx942 -I/root/.aiter/build/pa_b218c05023abc4cd4f3fce7059bf4982/include -c p
a_b218c05023abc4cd4f3fce7059bf4982.cpp -o pa_b218c05023abc4cd4f3fce7059bf4982.o                                                                                                                                                                                                                   
In file included from pa_b218c05023abc4cd4f3fce7059bf4982.cpp:1:                                                                                                                                                                                                                                  
In file included from /root/.aiter/build/pa_b218c05023abc4cd4f3fce7059bf4982/include/pa.cuh:26:                                                                                                                                                                                                   
In file included from /root/.aiter/build/pa_b218c05023abc4cd4f3fce7059bf4982/include/quant_utils.cuh:19:                                                                                                                                                                                          
/root/.aiter/build/pa_b218c05023abc4cd4f3fce7059bf4982/include/vec_convert.h:66:1: warning: non-void function does not return a value [-Wreturn-type]                                                                                                                                             
   66 | }                                                                                                                                                                                                                                                                                         
      | ^                                                                                                                                                                                                                                                                                         
/root/.aiter/build/pa_b218c05023abc4cd4f3fce7059bf4982/include/vec_convert.h:75:1: warning: non-void function does not return a value [-Wreturn-type]                                                                                                                                             
   75 | }                                                                                                                                                                                                                                                                                         
      | ^                                                                                                                                                                                                                                                                                         
/root/.aiter/build/pa_b218c05023abc4cd4f3fce7059bf4982/include/vec_convert.h:84:1: warning: non-void function does not return a value [-Wreturn-type]                                                                                                                                             
   84 | }                                                                                                                                                                                                                                                                                         
      | ^                                                                                                                                                                                                                                                                                         
In file included from pa_b218c05023abc4cd4f3fce7059bf4982.cpp:1:                                                                                                                                                                                                                                  
/root/.aiter/build/pa_b218c05023abc4cd4f3fce7059bf4982/include/pa.cuh:48:9: warning: 'DIVIDE_ROUND_UP' macro redefined [-Wmacro-redefined]                                                                                                                                                        
   48 | #define DIVIDE_ROUND_UP(a, b) (((a) + (b) - 1) / (b))                                                                                                                                                                                                                                     
      |         ^                                                                                                                                                                                                                                                                                 
/root/.aiter/build/pa_b218c05023abc4cd4f3fce7059bf4982/include/pa_common.cuh:26:9: note: previous definition is here                                                                                                                                                                              
   26 | #define DIVIDE_ROUND_UP(a, b) (((a) + (b)-1) / (b))                                                                                                                                                                                                                                       
      |         ^                                                                                                                                                                                                                                                                                 
4 warnings generated when compiling for gfx942.                                                                                                                                                                                                                                                   
In file included from pa_b218c05023abc4cd4f3fce7059bf4982.cpp:1:                                                                                                                                                                                                                                  
In file included from /root/.aiter/build/pa_b218c05023abc4cd4f3fce7059bf4982/include/pa.cuh:26:                                                                                                                                                                                                   
In file included from /root/.aiter/build/pa_b218c05023abc4cd4f3fce7059bf4982/include/quant_utils.cuh:19:
/root/.aiter/build/pa_b218c05023abc4cd4f3fce7059bf4982/include/vec_convert.h:66:1: warning: non-void function does not return a value [-Wreturn-type]
   66 | }
      | ^
/root/.aiter/build/pa_b218c05023abc4cd4f3fce7059bf4982/include/vec_convert.h:75:1: warning: non-void function does not return a value [-Wreturn-type]
   75 | }
      | ^
/root/.aiter/build/pa_b218c05023abc4cd4f3fce7059bf4982/include/vec_convert.h:84:1: warning: non-void function does not return a value [-Wreturn-type]
   84 | }
      | ^
In file included from pa_b218c05023abc4cd4f3fce7059bf4982.cpp:1:
/root/.aiter/build/pa_b218c05023abc4cd4f3fce7059bf4982/include/pa.cuh:48:9: warning: 'DIVIDE_ROUND_UP' macro redefined [-Wmacro-redefined]
   48 | #define DIVIDE_ROUND_UP(a, b) (((a) + (b) - 1) / (b))
      |         ^
/root/.aiter/build/pa_b218c05023abc4cd4f3fce7059bf4982/include/pa_common.cuh:26:9: note: previous definition is here

@tjtanaa
Copy link
Contributor

tjtanaa commented Jul 11, 2025

This fix works. Thank you for the quick fix. 🚀

@valarLip valarLip self-assigned this Jul 11, 2025
Copy link
Collaborator

@valarLip valarLip left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@valarLip valarLip merged commit 916bf3c into main Jul 12, 2025
11 of 12 checks passed
@valarLip valarLip deleted the pa_fix2 branch July 12, 2025 11:09
cagrikymk pushed a commit that referenced this pull request Jul 30, 2025
* fix compile bug

* add aiter debug

* add mp lock to compile process

* format code

---------

Co-authored-by: valarLip <103567126+valarLip@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants