[PATCH] D83818: [AArch64] fjcvtzs,rmif,cfinv,setf* all clobber nzcv

Fri Jul 24 09:41:37 PDT 2020

jroelofs marked an inline comment as done.
jroelofs added inline comments.

================
Comment at: llvm/test/CodeGen/AArch64/fjcvtzs.mir:1
+# RUN: llc -o - %s -mtriple=arm64-eabi -mattr=+jsconv -run-pass=legalizer
+
----------------
SjoerdMeijer wrote:
> I am very sorry, looking at this, I am changing my mind on this. This test is not testing anything, there are no CHECK tags. And in fact, what you had before is better and what we want: an llc test fed with an .ll, using stop-after to produce mir and check for implicit defs/uses.
> 
No worries. It's good for me to learn new tools!

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83818