[PATCH] D98932: [RISCV] Don't call setHasMultipleConditionRegisters(), so icmp is sunk

Evandro Menezes via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 30 07:14:05 PDT 2021


evandro added inline comments.


================
Comment at: llvm/test/CodeGen/RISCV/sink-icmp.ll:1
+; NOTE: Assertions have been autogenerated by utils/update_llc_test_checks.py
+; RUN: llc -mtriple=riscv32 -verify-machineinstrs < %s \
----------------
jrtc27 wrote:
> Could you please show the diff to this test?
@philipp.tomsich, please commit this test before the change so that this patch shows just its improvements to it.  Other than other comments, it LGTM.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D98932



More information about the llvm-commits mailing list