[llvm] [GlobalIsel] Add G_SCMP and G_UCMP instructions (PR #98894)

Michael Maitland via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 15 09:14:39 PDT 2024


================
@@ -1,4 +1,5 @@
-# RUN: not --crash llc -verify-machineinstrs -run-pass none -o /dev/null %s 2>&1 | FileCheck %s
+# RUN: not --crash llc -verify-machineinstrs -mtriple=arm64 -run-pass none -o /dev/null %s 2>&1 | FileCheck %s
+# REQUIRES: aarch64-registered-target
----------------
michaelmaitland wrote:

I am building targets `AArch64;AMDGPU;RISCV` and see no test failures when I run `ninja check-llvm-machineverifier` using Release build on `main` or when I checkout this branch but revert this test case. Do I need to do something else to reproduce? 

I am surprised that it is failing since there we have seen no issue since the beginning of March. If this really is a problem, I think it deserves to be split into a separate PR. I also wonder why we have not caught this sooner.

https://github.com/llvm/llvm-project/pull/98894


More information about the llvm-commits mailing list