Skip to content

Commit 19fdc9a

Browse files
authored
[Bugfix] Fix header include issue in rope (#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 <[email protected]>
1 parent 03ca2b2 commit 19fdc9a

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)