<div dir="ltr">I agree with Joerg. I don't really see much value in disabling idiom recognition. If the target isn't lowering memset/memcpy well, seems that fixing that would be more beneficial than being able to disable idiom recognition.<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Aug 16, 2017 at 10:22 AM, Joerg Sonnenberger via llvm-dev <span dir="ltr"><<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Tue, Aug 15, 2017 at 09:37:58PM -0700, Tim Northover via llvm-dev wrote:<br>
> The best advice is to file a bug report about the situation you're<br>
> seeing where a call to memset is bad for performance. There's clearly<br>
> something going wrong with Clang's heuristics and the best solution is<br>
> to fix that.<br>
<br>
</span>Do you mean clang or the backend? The discussion elsewhere about<br>
disabling memcpy intrinsic forming already makes me suspect that some<br>
targets don't handle those intrinsics well enough.<br>
<br>
Joerg<br>
<div class="HOEnZb"><div class="h5">______________________________<wbr>_________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/llvm-dev</a><br>
</div></div></blockquote></div><br></div>