<div dir="ltr">This source was actually removed back in October (<a href="https://android-review.googlesource.com/c/platform/frameworks/av/+/507575" target="_blank" class="cremed">https://android-review.<wbr>googlesource.com/c/platform/<wbr>frameworks/av/+/507575</a>). I think it might perhaps be time to upgrade the AOSP sources that the bots are using. Pengxuan, is this something that you can handle? aosp/master should be quite stable these days. I can help if you let me know about any problems you encounter.<div><br></div><div>Thanks,</div><div>Steve</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Nov 27, 2017 at 12:13 PM, Friedman, Eli <span dir="ltr"><<a href="mailto:efriedma@codeaurora.org" target="_blank">efriedma@codeaurora.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div text="#000000" bgcolor="#FFFFFF">
<div class="m_-7993290813151279682moz-cite-prefix">Not sure who the right contact is on
the Android side; adding Stephen Hines.<br>
<br>
In the meantime, we can hack up the source code on the bot, I
guess... adding Pengxuan.<span class="HOEnZb"><font color="#888888"><br>
<br>
-Eli</font></span><div><div class="h5"><br>
<br>
On 11/23/2017 2:32 AM, Oliver Stannard wrote:<br>
</div></div></div><div><div class="h5">
<blockquote type="cite">
<div class="m_-7993290813151279682WordSection1">
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d">Hi
Eli,<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d">The
instructions that are failing in the build are the ones that
I removed support for in this patch. They are pre-UAL
syntax, and we don’t support this generally, so I think the
long-term solution is to fix the code to use the UAL syntax
instead.<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d">Do
you know who I should contact in AOSP about getting that
code updated?<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d">The
alternative would be to revert this and keep these aliases,
but I’d rather not do that for the reasons discussed in
<a href="https://reviews.llvm.org/D39196" target="_blank">https://reviews.llvm.org/<wbr>D39196</a>.<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d">Oliver<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d"><u></u> <u></u></span></p>
<div style="border:none;border-left:solid blue 1.5pt;padding:0cm 0cm 0cm 4.0pt">
<div>
<div style="border:none;border-top:solid #b5c4df 1.0pt;padding:3.0pt 0cm 0cm 0cm">
<p class="MsoNormal"><b><span style="font-size:10.0pt;font-family:"Tahoma","sans-serif";color:windowtext" lang="EN-US">From:</span></b><span style="font-size:10.0pt;font-family:"Tahoma","sans-serif";color:windowtext" lang="EN-US"> Friedman, Eli
[<a class="m_-7993290813151279682moz-txt-link-freetext" href="mailto:efriedma@codeaurora.org" target="_blank">mailto:efriedma@codeaurora.<wbr>org</a>]
<br>
<b>Sent:</b> 22 November 2017 19:34<br>
<b>To:</b> Oliver Stannard;
<a class="m_-7993290813151279682moz-txt-link-abbreviated" href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
<b>Subject:</b> Re: [llvm] r318777 - [ARM] Remove
pre-UAL FLDM/FSTM aliases<u></u><u></u></span></p>
</div>
</div>
<p class="MsoNormal"><u></u> <u></u></p>
<div>
<p class="MsoNormal">On 11/21/2017 8:20 AM, Oliver Stannard
via llvm-commits wrote:<u></u><u></u></p>
</div>
<blockquote style="margin-top:5.0pt;margin-bottom:5.0pt">
<pre>Author: olista01<u></u><u></u></pre>
<pre>Date: Tue Nov 21 08:20:25 2017<u></u><u></u></pre>
<pre>New Revision: 318777<u></u><u></u></pre>
<pre><u></u> <u></u></pre>
<pre>URL: <a href="http://llvm.org/viewvc/llvm-project?rev=318777&view=rev" target="_blank">http://llvm.org/viewvc/llvm-<wbr>project?rev=318777&view=rev</a><u></u><u></u></pre>
<pre>Log:<u></u><u></u></pre>
<pre>[ARM] Remove pre-UAL FLDM/FSTM aliases<u></u><u></u></pre>
<pre><u></u> <u></u></pre>
<pre>These are pre-UAL syntax, and we don't support any other pre-UAL instructions,<u></u><u></u></pre>
<pre>with the exception of FLDMX/FSTMX, which don't have a UAL equivalent. Therefore<u></u><u></u></pre>
<pre>there's no reason to keep them or their AsmParser hacks around.<u></u><u></u></pre>
<pre><u></u> <u></u></pre>
<pre>With the AsmParser hacks removed, the FLDMX and FSTMX instructions get the same<u></u><u></u></pre>
<pre>operand diagnostics as the UAL instructions.<u></u><u></u></pre>
<pre><u></u> <u></u></pre>
<pre>Differential revision: <a href="https://reviews.llvm.org/D39196" target="_blank">https://reviews.llvm.org/<wbr>D39196</a><u></u><u></u></pre>
</blockquote>
<p class="MsoNormal"><br>
This is causing a failure on the polly AOSP buildbot (<a href="http://lab.llvm.org:8011/builders/aosp-O3-polly-before-vectorizer-unprofitable/builds/336/steps/build-aosp/logs/stdio" target="_blank">http://lab.llvm.org:8011/<wbr>builders/aosp-O3-polly-before-<wbr>vectorizer-unprofitable/<wbr>builds/336/steps/build-aosp/<wbr>logs/stdio</a>).
Error message:<br>
<br>
<br>
<u></u><u></u></p>
<pre><span class="m_-7993290813151279682stdout"><span style="font-size:13.5pt">frameworks/av/media/<wbr>libstagefright/codecs/aacenc/<wbr>src/asm/ARMV7/PrePostMDCT_v7.<wbr>s:31:2: error: invalid instruction<u></u><u></u></span></span></pre>
<pre><span class="m_-7993290813151279682stdout"><span style="font-size:13.5pt"> fstmfdd sp!, {d8 - d15}<u></u><u></u></span></span></pre>
<pre><span class="m_-7993290813151279682stdout"><span style="font-size:13.5pt"> ^<u></u><u></u></span></span></pre>
<p class="MsoNormal">-Eli<br>
<br>
<u></u><u></u></p>
<pre>-- <u></u><u></u></pre>
<pre>Employee of Qualcomm Innovation Center, Inc.<u></u><u></u></pre>
<pre>Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project<u></u><u></u></pre>
</div>
</div>
IMPORTANT NOTICE: The contents of this email and any attachments
are confidential and may also be privileged. If you are not the
intended recipient, please notify the sender immediately and do
not disclose the contents to any other person, use it for any
purpose, or store or copy the information in any medium. Thank
you.
</blockquote>
<p><br>
</p>
<pre class="m_-7993290813151279682moz-signature" cols="72">--
Employee of Qualcomm Innovation Center, Inc.
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project</pre>
</div></div></div>
</blockquote></div><br></div>