[PATCH] D71030: Tighten the check for Tool name ( ar, nm, dlltool, lib)

[Fangrui Song via Phabricator via llvm-commits]

MaskRay added a comment.

On Debian, the package "llvm-8" installs llvm-ar at `/usr/bin/llvm-ar-8`. Maybe we should check either `endswith("ar")` or `contains("ar-")`? I think the existing `_lower` part is probably not necessary.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71030