[PATCH] D12969: Don't raise inexact when lowering ceil, floor, round, trunc

[Tim Northover via llvm-commits]

t.p.northover added a comment.

Thanks for updating the patch Steve. One more thing I spotted:


================
Comment at: lib/Target/AArch64/AArch64InstrInfo.td:2488-2492
@@ -2487,7 +2507,2 @@
 
-// FRINTX is inserted to set the flags as required by FENV_ACCESS ON behavior
-// in the C spec. Setting hasSideEffects ensures it is not DCE'd.
-// <rdar://problem/13715968>
-// TODO: We should really model the FPSR flags correctly. This is really ugly.
-let hasSideEffects = 1 in {
 defm FRINTX : SingleOperandFPData<0b1110, "frintx", frint>;
----------------
Sorry for only bringing this up now, but this doesn't look right. I think if an FRINTX does get created it probably should be treated specially. You'd mostly only do so (by calling "rint") if you actually cared about the otherwise unmodeled FPSR flags, wouldn't you?


http://reviews.llvm.org/D12969