[Build] Use C++17 Constructor for tiktoken.cpp when C++20 is unavailable

[python3kgae]

The basic_string_view constructor:

template< class It, class End >
constexpr basic_string_view( It first, End last );

requires C++20.

To allow the code to compile with C++17, use the basic_string_view constructor:

constexpr basic_string_view( const CharT* s, size_type count );

when C++20 is not available

For #4661

[pytorch-bot]

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/5025

• 📄 Preview Python docs built from this PR

Note: Links to docs will display an error until the docs builds have been completed.

✅ No Failures

As of commit baa7067 with merge base 37cad01 ():
💚 Looks good so far! There are no failures yet. 💚

This comment was automatically generated by Dr. CI and updates every 15 minutes.

[cccclai]

Seems fine to me, will wait for CI signal

[larryliu0820]

Can you click the "Squash and merge" button with a good commit message? We are testing out landing PRs from GH.

[python3kgae]

Can you click the "Squash and merge" button with a good commit message? We are testing out landing PRs from GH.

Don't have write access :(
Here's the commit message.

[Build] Use C++17 Constructor for basic_string_view in tiktoken.cpp when C++20 is unavailable

The basic_string_view constructor:

template< class It, class End >
constexpr basic_string_view( It first, End last );

requires C++20.

To allow the code to compile with C++17, use the basic_string_view constructor:

constexpr basic_string_view( const CharT* s, size_type count );

when C++20 is not available

For #4661

[kirklandsign]

Can you click the "Squash and merge" button with a good commit message? We are testing out landing PRs from GH.

Don't have write access :( Here's the commit message.

[Build] Use C++17 Constructor for basic_string_view in tiktoken.cpp when C++20 is unavailable

The basic_string_view constructor:

template< class It, class End >
constexpr basic_string_view( It first, End last );

requires C++20.

To allow the code to compile with C++17, use the basic_string_view constructor:

constexpr basic_string_view( const CharT* s, size_type count );

when C++20 is not available

For #4661

Thank you for the feedback! We will update the rules on our side and let you know

[kirklandsign]

Seems that currently we need reviewers to merge the commit. I will do it.