[PATCH] D73201: [FPEnv][AArch64] Don't expand STRICT_FP_ROUND to an illegal truncating store

Wed Jan 22 10:05:14 PST 2020

john.brawn added a comment.

In D73201#1834254 <https://reviews.llvm.org/D73201#1834254>, @craig.topper wrote:

> So does AArch64 set FP_ROUND to Legal or Custom? STRICT_FP_ROUND should do the same.

FP_ROUND is Custom in AArch64. I could add something to the AArch64 target to handle this, but it's a bit odd that the default behaviour is for an invalid operation to be generated (and I see that -mtriple=sparc hits the same assertion failure, as it also has FP_ROUND set to Custom).

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D73201