<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Fri, Feb 2, 2018 at 4:23 PM Chandler Carruth <<a href="mailto:chandlerc@google.com">chandlerc@google.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_quote"><div dir="ltr">On Fri, Feb 2, 2018 at 4:03 PM David Woodhouse <<a href="mailto:dwmw2@infradead.org" target="_blank">dwmw2@infradead.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Thu, 2018-02-01 at 10:10 +0100, Hans Wennborg via llvm-dev wrote:<br>
><br>
> I saw the retpoline mitigation landed in r323155. Are we ready to<br>
> merge this to 6.0, or are there any open issues that we're waiting<br>
> for? Also, were there any followups I should know about? Also,<br>
> release notes please :-)<br>
<br>
Eep, please can we keep the command line option for clang and the thunk<br>
ABI matching what GCC, the Linux kernel and Xen are all doing?<br>
<br>
To say that I am not stunningly keen on<br>
<a href="https://lkml.org/lkml/2018/2/2/975" rel="noreferrer" target="_blank">https://lkml.org/lkml/2018/2/2/975</a> would be a bit of an<br>
understatement...</blockquote></div></div></blockquote><div><br></div><div>A minor note on this specific patch:</div><div><br></div><div>You don't need '-mretpoline -mretpoline-external-thunk'. The second flag implies the first. (Or should, if not, its a bug.) Our goal was that you needed exactly one flag to enable this in whatever form.</div></div></div>