<div dir="ltr">LGTM, although I suspect others' opinions are more relevant than mine...</div><br><div class="gmail_quote"><div dir="ltr">On Mon, Apr 25, 2016 at 2:08 AM Craig Topper <<a href="mailto:craig.topper@gmail.com">craig.topper@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">I'd like to change the default operation action for CTLZ_ZERO_UNDEF and CTTZ_ZERO_UNDEF to Expand in TargetLoweringBase. This will make LegalizeDAG automatically turn them into CTLZ and CTTZ.<div><br></div><div>For the in-tree targets, only AMDGPU and X86 have cases where these instructions should be the current default value of Legal. There a few cases where targets set them to Custom. But most targets set them to Expand to have LegalizeDAG convert them.</div><div><br></div><div>This change would allow us to remove all the setOperationAction calls to make them Expand on most targets and adds a few calls to setOperationAction on X86 and AMDGPU to make them Legal again for certain cases.</div><div><br></div><div>This could affect out of tree targets, but its easy to fix any out of tree targets that rely on the current default Legal value. Any out of tree targets that were already setting them to Expand will to continue to work, but just have redundant code.</div><div><br></div><div>Any objections?</div></div><div dir="ltr"><div><br clear="all"><div><br></div>-- <br><div>~Craig</div>
</div></div></blockquote></div>