Skip to content

Commit 6db8bc7

Browse files
ganyi1996ppoyangxiaojun0126
authored andcommitted
[Bugfix] Fix header include issue in rope (vllm-project#2397)
### What this PR does / why we need it? vLLM-Ascend's rope implementaion include several header file that are not supposed to be included by outside users. Current implementation may break when canntoolkits update, this PR remove those not compatible file includes to guarantee the safety of upgrading cann toolkits. ### Does this PR introduce _any_ user-facing change? No ### How was this patch tested? Tested by rope unittest - vLLM version: v0.10.0 - vLLM main: vllm-project/vllm@3e6dd40 Signed-off-by: ganyi <pleaplusone.gy@gmail.com>
1 parent a6e0a9a commit 6db8bc7

File tree

1 file changed

+0
-5
lines changed

1 file changed

+0
-5
lines changed

csrc/kernels/pos_encoding_kernels.cpp

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -15,11 +15,6 @@
1515
*/
1616

1717
#include "kernel_operator.h"
18-
#include "kernel_tpipe_impl.h"
19-
#include "kernel_tensor_impl.h"
20-
#include "kernel_type.h"
21-
#include "kernel_operator_intf.h"
22-
#include "inner_interface/inner_kernel_operator_intf.h"
2318
#include <stdio.h>
2419
#include "types.h"
2520
#include "utils.h"

0 commit comments

Comments
 (0)