<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Tue, Jun 23, 2015 at 12:03 PM Akira Hatanaka <<a href="mailto:ahatanak@gmail.com">ahatanak@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Jun 22, 2015 at 2:00 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"><div dir="ltr"><br><br><div class="gmail_quote"><span><div dir="ltr">On Mon, Jun 22, 2015 at 1:49 PM Akira Hatanaka <<a href="mailto:ahatanak@gmail.com" target="_blank">ahatanak@gmail.com</a>> wrote:<br></div><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">In <a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__reviews.llvm.org_D10414-23191915&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=BSqEv9KvKMW_Ob8SyngJ70KdZISM_ASROnREeq0cCxk&m=hO4s0j_qRWgIche0jG-a7QHL0HJGwsxn-sODhOhZXmo&s=VdmyU4VpGauPFGvYx-JyyeN8P0i8O63x5rHam1X2MZs&e=" rel="noreferrer" target="_blank">http://reviews.llvm.org/D10414#191915</a>, @echristo wrote:<br>
<br>
> Another possibility is that this gets mapped as a normal command line option passed into cc1 and then it can just be a subtarget feature?<br>
<br>
<br>
Would it look something like this in the IR?<br>
<br>
"target-features"="+neon,+vfp3,arn-restrict-it=false"<br></blockquote><div><br></div></span><div>was thinking of more just:</div><div><br></div><div>"target-features"="+neon,+vfp3,-restrict-it"</div><div><br></div><div>Though if we're going for "restrict-it" blocks perhaps a better name that works well with +/-?</div><div><br></div><div>I've generally stayed away from turning command line options into features, but in the case of something that appears to turn on actual code generation features it seems like it might be reasonable.</div><span><font color="#888888"><div><br></div></font></span></div></div></blockquote><div><br></div></div></div></div><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div>Since restrict-it is a tri-state, I think you'll have to define two new subtarget features in ARM.td (see the attached patch). I'm not sure how prefixing "restrict-it" by "-" is going to work.</div></div></div></div></blockquote><div><br></div><div>I think default is a red herring here a bit. It's either on or off right now, we might have to bake in some back end knowledge in the front end about when to apply it, but it should still work out yes?</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><br></div><div>This approach can potentially create larger numbers of subtarget objects as two subtarget objects have to be created If one function has feature "restricted-it" and another has feature "no-restricted-it". If we take the approach in my original patch, we need only one subtarget object. Do you think that's going to be a problem?<br></div></div></div></div><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><br></div></div></div></div></blockquote><div><br></div><div>Irrespective of the above I'm not too worried about this. In general, people have a tendency to use the same command line options everywhere, especially for this sort of thing.</div><div><br></div><div>-eric</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div></div><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"><div dir="ltr"><div class="gmail_quote"><span><font color="#888888"><div></div><div>-eric</div></font></span><span><div> </div><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">
<br>
<br>
<a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__reviews.llvm.org_D10414&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=BSqEv9KvKMW_Ob8SyngJ70KdZISM_ASROnREeq0cCxk&m=hO4s0j_qRWgIche0jG-a7QHL0HJGwsxn-sODhOhZXmo&s=fxP0WmNt0R-KiwPr0pUDY_vf3U05lVZ26pFId77yvcc&e=" rel="noreferrer" target="_blank">http://reviews.llvm.org/D10414</a><br>
<br>
EMAIL PREFERENCES<br>
  <a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__reviews.llvm.org_settings_panel_emailpreferences_&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=BSqEv9KvKMW_Ob8SyngJ70KdZISM_ASROnREeq0cCxk&m=hO4s0j_qRWgIche0jG-a7QHL0HJGwsxn-sODhOhZXmo&s=WVGN99uXi7L9TvD5_yNbuv9E_75ETfZPpv7tcJoAudM&e=" rel="noreferrer" target="_blank">http://reviews.llvm.org/settings/panel/emailpreferences/</a><br>
<br>
<br>
</blockquote></span></div></div>
</blockquote></div></div></div></blockquote></div></div>