Skip to content

Commit 6e58ede

Browse files
committed
Update on "Fix test_llama_runner by hiding tiktoken"
Summary: We don't always want to build tiktoken dependencies (re2 and abseil) so this PR only build it if the option is on. Test Plan: Reviewers: Subscribers: Tasks: Tags: Differential Revision: [D56178928](https://our.internmc.facebook.com/intern/diff/D56178928) [ghstack-poisoned]
1 parent 5d2fac8 commit 6e58ede

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

examples/models/llama2/tokenizer/test/targets.bzl

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -8,12 +8,12 @@ def define_common_targets():
88
"""
99

1010
runtime.cxx_test(
11-
name = "test",
11+
name = "test_bpe_tokenizer",
1212
srcs = [
13-
"test_tokenizer.cpp",
13+
"test_bpe_tokenizer.cpp",
1414
],
1515
deps = [
16-
"//executorch/examples/models/llama2/tokenizer:tokenizer",
16+
"//executorch/examples/models/llama2/tokenizer:bpe_tokenizer",
1717
],
1818
env = {
1919
"RESOURCES_PATH": "$(location :resources)/resources",
@@ -26,7 +26,7 @@ def define_common_targets():
2626
"test_tiktoken.cpp",
2727
],
2828
deps = [
29-
"//executorch/examples/models/llama2/tokenizer:tokenizer",
29+
"//executorch/examples/models/llama2/tokenizer:tiktoken",
3030
],
3131
env = {
3232
"RESOURCES_PATH": "$(location :resources_fb_only)/resources",

0 commit comments

Comments
 (0)