[PATCH] D73194: [FPEnv][ARM] Add lowering of STRICT_FSETCC and STRICT_FSETCCS

[John Brawn via Phabricator via llvm-commits]

john.brawn added a comment.

In D73194#1849170 <https://reviews.llvm.org/D73194#1849170>, @dmgreen wrote:

> Is it possible for arm_cmpfpe have both a chain and a glue? I presume it's the cmp that would need the chain, not anything else that gets expanded?


Doing this gets an assertion failure during instruction selection:

  Assertion `NumMIOperands >= II.getNumOperands() && NumMIOperands <= II.getNumOperands() + II.getNumImplicitDefs() + NumImpUses && "#operands for dag node doesn't match .td file!"'

VCMPES has one implicit def (FPSCR_NZCV), so having either chain or glue the numbers balance out, but if you have both you have one more operand than what's expected and you get the assertion.


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

https://reviews.llvm.org/D73194