<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><br class=""><div><blockquote type="cite" class=""><div class="">On Jan 29, 2015, at 6:01 PM, Michael Zolotukhin <<a href="mailto:mzolotukhin@apple.com" class="">mzolotukhin@apple.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><meta http-equiv="Content-Type" content="text/html charset=utf-8" class=""><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><br class=""><div class=""><blockquote type="cite" class=""><div class="">On Jan 29, 2015, at 4:21 PM, Chandler Carruth <<a href="mailto:chandlerc@google.com" class="">chandlerc@google.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div dir="ltr" class="">Do we use the context instruction to extract fast math flags elsewhere?<div class=""><br class=""></div><div class="">I wouldn't expect that to be reliably at all. Nothing *requires* the context instruction to be the one governing the semantics.</div><div class=""><br class=""></div><div class="">I think we need to pass the fast math flags directly into SimplifyBinOp for this to work.</div></div></div></blockquote><div class=""><br class=""></div><div class="">I tried to avoid adding a new argument, but you are right: using the context instruction isn’t appropriate for this.</div><div class=""><br class=""></div><div class="">Updated patch is attached, is it better?</div></div></div></div></blockquote>Oops, I forgot the patch. Here it is:</div><div><br class=""></div><div></div></body></html>