<html><head><meta http-equiv="content-type" content="text/html; charset=utf-8"></head><body dir="auto"><div><br><br>Sent from my iPad</div><div><br>On Sep 11, 2013, at 11:16 PM, Owen Anderson <<a href="mailto:resistor@mac.com">resistor@mac.com</a>> wrote:<br><br></div><blockquote type="cite"><div><meta http-equiv="Content-Type" content="text/html charset=windows-1252"><br><div><div>On Sep 11, 2013, at 4:24 PM, Arnold Schwaighofer <<a href="mailto:aschwaighofer@apple.com">aschwaighofer@apple.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><span style="font-family: Helvetica; font-size: medium; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; display: inline !important; float: none; ">CC’ing Owen who I have been told might have started this at one point. Owen is my thinking that threading fast-math flags into ISel is a bigger undertaking a correct estimate or is it really just “creating and maintaining a BinOp->FastMathFlag map”? I am guessing the maintaining part might be hairy.</span></blockquote></div><br><div>I would love to propagate fast-math flags through ISel and kill off the FastMathFlag, but it'll be a non-trivial amount of work to update every DAGCombine to respect them properly.</div></div></blockquote><div><br></div><div>Fast math is not the only bit of information that's lost. There is also nsw and nuw which would have been useful for isel as well. That said I <span style="-webkit-tap-highlight-color: rgba(26, 26, 26, 0.296875); -webkit-composition-fill-color: rgba(175, 192, 227, 0.230469); -webkit-composition-frame-color: rgba(77, 128, 180, 0.230469); ">agree it's not worthwhile to add more complexity to sdisel. We should add them after we migrate to MI based isel. </span><div><br></div><div>Evan</div><div><br><blockquote type="cite"><div><div><br></div><div>--Owen</div></div></blockquote><blockquote type="cite"><div><span>_______________________________________________</span><br><span>llvm-commits mailing list</span><br><span><a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a></span><br><span><a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a></span><br></div></blockquote></div></div></body></html>