[libcxx-commits] [PATCH] D143007: [libc++][CI] Uses LLVM 17 in Docker.

[Joe Loser via Phabricator via libcxx-commits]

jloser added a comment.

In D143007#4103560 <https://reviews.llvm.org/D143007#4103560>, @ldionne wrote:

> Is `LLVM_HEAD_VERSION=17` going to work yet?

I don't think so based on looking at the code:

  declare -A LLVM_VERSION_PATTERNS
  LLVM_VERSION_PATTERNS[9]="-9"
  LLVM_VERSION_PATTERNS[10]="-10"
  LLVM_VERSION_PATTERNS[11]="-11"
  LLVM_VERSION_PATTERNS[12]="-12"
  LLVM_VERSION_PATTERNS[13]="-13"
  LLVM_VERSION_PATTERNS[14]="-14"
  LLVM_VERSION_PATTERNS[15]="-15"
  LLVM_VERSION_PATTERNS[16]="-16"
  LLVM_VERSION_PATTERNS[17]=""
  
  if [ ! ${LLVM_VERSION_PATTERNS[$LLVM_VERSION]+_} ]; then
      echo "This script does not support LLVM version $LLVM_VERSION"
      exit 3
  fi

since `LLVM_VERSION_PATTERNS[17]` is an empty string.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D143007/new/

https://reviews.llvm.org/D143007