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

Fri Nov 6 01:33:26 PST 2020

yroux added a comment.
Herald added a subscriber: frasercrmck.

Hi David,

Now that ARMv7 bots are back online after some infra issues, I see that asrm_function_name.ll is failling because tries to generate 64bit arm code but the compiler is not configured with AArch64 target on these bots.

bot logs are available here: http://lab.llvm.org:8011/#/builders/26/builds/328

================
Comment at: llvm/test/tools/UpdateTestChecks/update_llc_test_checks/Inputs/arm_function_name.ll:3
+;
+; RUN: llc -mtriple=arm64-unknown-linux < %s | FileCheck --prefi=LINUX %s
+; RUN: llc -mtriple=armv7-apple-darwin < %s | FileCheck --prefix=DARWIN %s
----------------
I think that it should be moved to aarch64_function_name.ll 

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82995