[PATCH] D82995: [UpdateTestChecks] Allow $ in function names

Jessica Clarke via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 17 11:06:37 PDT 2020


jrtc27 added a comment.

In D82995#2279840 <https://reviews.llvm.org/D82995#2279840>, @alanphipps wrote:

> There exist downstream ARM compilers that doesn't support ARM64 and fail with Inputs/arm_function_name.ll.  You should consider guarding arm64 specific tests.

arm64 is an alias for aarch64, that should just be armv7 like the rest. Also the aarch64 test should probably gain an IOS test, it's weird that the arm one has one but not the aarch64 one.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82995



More information about the llvm-commits mailing list