[PATCH] D115808: [DAGCombiner] Avoid combining adjacent stores at -O0 to improve debug experience

Shivam Gupta via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 15 09:52:56 PST 2021


xgupta added inline comments.


================
Comment at: llvm/test/CodeGen/RISCV/dbg-combine.ll:1
+; NOTE: Assertions have been autogenerated by utils/update_llc_test_checks.py
+; RUN: llc -mtriple=riscv64 -verify-machineinstrs < %s | FileCheck %s
----------------
jrtc27 wrote:
> This is way too complicated a test case. All you need is IR that makes the same store twice and verify the codegen has the two stores in it.
> 
> It's also clearly *not* currently using utils/update_llc_test_checks.py.
Yes, I have tried, but check lines are not generated. I will look tomorrow at how to make that script work and think of reducing test case.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115808



More information about the llvm-commits mailing list