[PATCH] D128433: [LoongArch] Add LoongArch support to update_llc_test_checks

[WÁNG Xuěruì via Phabricator via llvm-commits]

xen0n added inline comments.


================
Comment at: llvm/test/tools/UpdateTestChecks/update_llc_test_checks/Inputs/loongarch_function_name.ll:2
+; Check that we accept functions with '$' in the name.
+;
+; RUN: llc -mtriple=loongarch32-unknown-linux < %s | FileCheck %s
----------------
MaskRay wrote:
> The convention does not add otherwise empty `;`
You mean that by convention this line should be removed? I grepped the nearby test cases but it seems most are actually including these `;` lines.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D128433