[PATCH] D143036: [RISCV] Add vendor-defined XTHeadBs (single-bit) extension

[Philipp Tomsich via Phabricator via llvm-commits]

philipp.tomsich added inline comments.


================
Comment at: llvm/test/CodeGen/RISCV/rv32xtheadbs.ll:1
+; NOTE: Assertions have been autogenerated by utils/update_llc_test_checks.py
+; RUN: llc -mtriple=riscv32 -verify-machineinstrs < %s \
----------------
reames wrote:
> In this diff, you have coverage for both 32 and 64, but is this extension actually defined for RV32?  The previous diff (THeadBa) was only tested on RV64.  Why are these two handled differently here?
> In this diff, you have coverage for both 32 and 64, but is this extension actually defined for RV32?  The previous diff (THeadBa) was only tested on RV64.  Why are these two handled differently here?

Test cases to THeadBa have been added in the latest revisison.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D143036