<div dir="ltr">Hi Renato,<div><br></div><div>We've always had two options for this sort of thing, just much more horribly implemented. :)</div><div><br></div><div>Basically the backend developers want to be able to override (largely for debugging purposes) the "default" for whether or not to construct IT blocks. It's just overriding the source level and it's good that we can do this without having developers overwrite the source themselves (it could be quite large).</div><div><br></div><div>I've delayed on reviewing this patch because I'm not quite sure how I want particular target options to be passed to the backend, but in general the idea of the patch ("allow the backend to override the IR") isn't a bad thing.</div><div><br></div><div>-eric</div><br><div class="gmail_quote"><div dir="ltr">On Fri, Jun 19, 2015 at 4:28 AM Renato Golin <<a href="mailto:renato.golin@linaro.org" target="_blank">renato.golin@linaro.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I'm confused. You removed the back-end option in Clang, but it remains here (IT), meaning you now have two identical command line options that do things slightly different. The function attributes set IT behaviour on every function and the backend option that sets it in a module level.<br>
<br>
This will create a confusing environment on which to develop things and I don't want to extend this pattern. If the option needs to be on a function basis, than we need to make sure that whatever generates IR does that on a function level, and free the backend from having to ponder about module level options.<br>
<br>
In summary, IT options must go. :)<br>
<br>
cheers,<br>
--renato<br>
<br>
<br>
<a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__reviews.llvm.org_D10416&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=mQ4LZ2PUj9hpadE3cDHZnIdEwhEBrbAstXeMaFoB9tg&m=k7vegRn7wMVTodC0F4ACyBqnnYAuYxD0QZc0s531lR8&s=lWjzbYjvNadlHwenTYV1U5a31SdfG6Gv9M6Ro8bjTXc&e=" rel="noreferrer" target="_blank">http://reviews.llvm.org/D10416</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=mQ4LZ2PUj9hpadE3cDHZnIdEwhEBrbAstXeMaFoB9tg&m=k7vegRn7wMVTodC0F4ACyBqnnYAuYxD0QZc0s531lR8&s=ziLbvnkO3vZslMjweKGEkthUnN7KiTOUFqfkRlxS43E&e=" rel="noreferrer" target="_blank">http://reviews.llvm.org/settings/panel/emailpreferences/</a><br>
<br>
<br>
</blockquote></div></div>