<div dir="auto">Thank you, Hans!</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Mar 25, 2019, 2:27 AM Hans Wennborg via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">This broke the build with GCC 4.8, see very long error message<br>
attached. I've committed r356875 to fix it.<br>
<br>
On Fri, Mar 22, 2019 at 6:21 PM Alina Sbirlea via llvm-commits<br>
<<a href="mailto:llvm-commits@lists.llvm.org" target="_blank" rel="noreferrer">llvm-commits@lists.llvm.org</a>> wrote:<br>
><br>
> Author: asbirlea<br>
> Date: Fri Mar 22 10:22:19 2019<br>
> New Revision: 356783<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=356783&view=rev" rel="noreferrer noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=356783&view=rev</a><br>
> Log:<br>
> [AliasAnalysis] Second prototype to cache BasicAA / anyAA state.<br>
><br>
> Summary:<br>
> Adding contained caching to AliasAnalysis. BasicAA is currently the only one using it.<br>
><br>
> AA changes:<br>
> - This patch is pulling the caches from BasicAAResults to AAResults, meaning the getModRefInfo call benefits from the IsCapturedCache as well when in "batch mode".<br>
> - All AAResultBase implementations add the QueryInfo member to all APIs. AAResults APIs maintain wrapper APIs such that all alias()/getModRefInfo call sites are unchanged.<br>
> - AA now provides a BatchAAResults type as a wrapper to AAResults. It keeps the AAResults instance and a QueryInfo instantiated to batch mode. It delegates all work to the AAResults instance with the batched QueryInfo. More API wrappers may be needed in BatchAAResults; only the minimum needed is currently added.<br>
><br>
> MemorySSA changes:<br>
> - All walkers are now templated on the AA used (AliasAnalysis=AAResults or BatchAAResults).<br>
> - At build time, we optimize uses; now we create a local walker (lives only as long as OptimizeUses does) using BatchAAResults.<br>
> - All Walkers have an internal AA and only use that now, never the AA in MemorySSA. The Walkers receive the AA they will use when built.<br>
><br>
> - The walker we use for queries after the build is instantiated on AliasAnalysis and is built after building MemorySSA and setting AA.<br>
> - All static methods doing walking are now templated on AliasAnalysisType if they are used both during build and after. If used only during build, the method now only takes a BatchAAResults. If used only after build, the method now takes an AliasAnalysis.<br>
><br>
> Subscribers: sanjoy, arsenm, jvesely, nhaehnle, jlebar, george.burgess.iv, llvm-commits<br>
><br>
> Tags: #llvm<br>
><br>
> Differential Revision: <a href="https://reviews.llvm.org/D59315" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D59315</a><br>
><br>
> Modified:<br>
>     llvm/trunk/include/llvm/Analysis/AliasAnalysis.h<br>
>     llvm/trunk/include/llvm/Analysis/BasicAliasAnalysis.h<br>
>     llvm/trunk/include/llvm/Analysis/CFLAndersAliasAnalysis.h<br>
>     llvm/trunk/include/llvm/Analysis/CFLSteensAliasAnalysis.h<br>
>     llvm/trunk/include/llvm/Analysis/GlobalsModRef.h<br>
>     llvm/trunk/include/llvm/Analysis/MemorySSA.h<br>
>     llvm/trunk/include/llvm/Analysis/ObjCARCAliasAnalysis.h<br>
>     llvm/trunk/include/llvm/Analysis/ScalarEvolutionAliasAnalysis.h<br>
>     llvm/trunk/include/llvm/Analysis/ScopedNoAliasAA.h<br>
>     llvm/trunk/include/llvm/Analysis/TypeBasedAliasAnalysis.h<br>
>     llvm/trunk/lib/Analysis/AliasAnalysis.cpp<br>
>     llvm/trunk/lib/Analysis/BasicAliasAnalysis.cpp<br>
>     llvm/trunk/lib/Analysis/CFLAndersAliasAnalysis.cpp<br>
>     llvm/trunk/lib/Analysis/GlobalsModRef.cpp<br>
>     llvm/trunk/lib/Analysis/MemorySSA.cpp<br>
>     llvm/trunk/lib/Analysis/ObjCARCAliasAnalysis.cpp<br>
>     llvm/trunk/lib/Analysis/ScalarEvolutionAliasAnalysis.cpp<br>
>     llvm/trunk/lib/Analysis/ScopedNoAliasAA.cpp<br>
>     llvm/trunk/lib/Analysis/TypeBasedAliasAnalysis.cpp<br>
>     llvm/trunk/lib/Target/AMDGPU/AMDGPUAliasAnalysis.cpp<br>
>     llvm/trunk/lib/Target/AMDGPU/AMDGPUAliasAnalysis.h<br>
>     llvm/trunk/unittests/Analysis/AliasAnalysisTest.cpp<br>
>     llvm/trunk/unittests/Analysis/BasicAliasAnalysisTest.cpp<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank" rel="noreferrer">llvm-commits@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote></div>