<div dir="ltr"><div class="gmail_extra">Yes, I that works for me. I've been working on rewriting the .td file, but I can do that later.</div><div class="gmail_extra"><br></div><div class="gmail_extra">There are other attributes that we need to check besides target-cpu and target-features:</div><div class="gmail_extra"><br></div><div class="gmail_extra">- Fast FP math attributes: I think we should reject inlining if the caller has a fast fp-math attribute (e.g., unsafe-fp-math = true) but the callee doesn't. Alternatively, we can change the caller's fp-math attribute (to unsafe-fp-math = false). I know the long term solution would be to model these attributes at the instruction-level, but until we make that change, we should treat them as function attributes.</div><div class="gmail_extra">- NoImplicitFloat.</div><div class="gmail_extra">- Probably we should check some of the target-specific attributes too.</div><div class="gmail_extra"><br></div><div class="gmail_extra"><div class="gmail_quote">On Tue, Apr 7, 2015 at 5:17 PM, Eric Christopher <span dir="ltr"><<a href="mailto:echristo@gmail.com" target="_blank">echristo@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Hi Akira,<br>
<br>
I think for right now since it's fairly trivial to come up with a testcase that will run into problems we should do a quick conservative patch to the inliner first that will enable some progress - i.e. we should check the target-cpu and target-features strings and if they don't match exactly go ahead and reject in the inliner. This is, obviously, much less than optimal, but will get us to an iterative solution.<br>
<br>
Thoughts?<br>
<span class=""><font color="#888888"><br>
-eric<br>
</font></span><div class=""><div class="h5"><br>
<br>
<a href="http://reviews.llvm.org/D7802" target="_blank">http://reviews.llvm.org/D7802</a><br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/settings/panel/emailpreferences/</a><br>
<br>
<br>
</div></div></blockquote></div><br></div></div>