jrtc27 added inline comments.
================ Comment at: llvm/test/CodeGen/RISCV/rv32Zbr.ll:1 +; NOTE: Assertions have been autogenerated by utils/update_llc_test_checks.py +; RUN: llc -mtriple=riscv32 -mattr=experimental-zbr -verify-machineinstrs < %s \ ---------------- craig.topper wrote: > jrtc27 wrote: > > Lowercase z in the names of these files as it's a (sort of) -march string > The bitmanip tests were already have all us "Zb*" in their file names. Should > we match that or rename the existing tests? :( ideally they'd be renamed IMO, but if not then yeah I guess this should stay as it is to match Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D99009/new/ https://reviews.llvm.org/D99009 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits