xen0n added a comment.

I don't know if the clang changes should be split into its own commit (or the 
title of this commit amended to mention `[Clang]` but I don't know if this is 
appropriate), but the rest looks reasonable.



================
Comment at: llvm/test/CodeGen/LoongArch/inline-asm-constraint-error.ll:2
+; RUN: not llc -mtriple=loongarch32 < %s 2>&1 | FileCheck %s
+; RUN: not llc -mtriple=loongarch64 < %s 2>&1 | FileCheck %s
+
----------------
double dashes for `--mtriple`


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D134157

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to