<div dir="ltr">  > Just to be clear, you don't think it has potential because it's been<br>> disgned into an inner-loop corner and would take extensive rewriting to<br>> handle OLV?<br><div><br></div><div>First of all, if one wants to do dependence analysis for OLV (which is what I'm currently working on), they may get away</div><div>with extending LAA, or writing their own OLV checker.</div><div><br></div><div>That said, no it doesn't seem that the current LAA can help. But that we can't support OLV currently</div><div>is not the core problem for me. We may do it eventually, but if the implementation is still hacky, then</div><div>it would still be bad I think. The core problem is that there's no clear theoretical foundation to support it.</div><div><br></div><div>The code seems to me like a set of "Oh, we needed to handle this case so we added an `if` there".</div><div>Which to me is not the correct way to go and it's pretty much the opposite of DA.</div><div><br></div><div>(Again, I don't mean to criticize LAA implementors - they probably had their reasons and I guess</div><div>they know way more than me)</div><div><br></div><div>> it certainly seems like extending DA is the way to go</div><div><br></div><div>Maybe but maybe not. Maybe a good theory for run-time checks ends up</div><div>being different than DA's ideas (this is partly what I have to do this summer).<br>In any case, extending </div><div><br></div><div>> but I'd like to hear from the current vectorizer<br>> maintainers because I don't have enough knowledge to make an informed<br>> judgment.<br><br>Me too!<br><br>> There's the VPlan infrastructure which I have not heard much about for<br>> several months.  What is going on with that?  Yes, that's a vector<br>> codegen issue but it may be useful to have a more complete picture of<br>> how all this works or will work.<br><br>Sorry, I don't know much about VPlan. I'm involved in the RV (<a href="https://github.com/cdl-saarland/rv">https://github.com/cdl-saarland/rv</a>) <br><br>> Note that when the development of LAA started it also did static checks only, even though DA already existed in the code base<br><br>Interesting, thanks.<br><br>> Thanks for sharing your analysis.<br><br>No problem :)<br><br>> I am not sure if that is an entirely fair characterization of LAA. LAA is being used by the vectorizer (and other passes) in production for a few years now. None of the in-tree users of DA seem to be enabled by default and therefore LAA probably has an order of magnitude more testing, bug fixes & tuning.<br><br>No argument there. The fact that it is used, doesn't necessarily mean it's clean nor that it has some strong theory supporting it.<br><br>> DA’s implementation might be cleaner, but as mentioned earlier, DA handles only a small subset of things LAA handles and hence I am not sure comparing the code-complexity is too helpful.<br><br>DA does not handle a small subset of LAA's checks, unless I miss something. It handles way more when it comes to static checking.<br>I think that comparing code complexity is important. DA is about double the size of LAA yet it's way more understandable. And the reason for that</div><div>I don't think it is that it does something more trivial. Rather, it's based on a clear paper and has clearly implemented it.<br><br></div>> IMO a lot of LAA complexity comes from things DA does not handle, in particular runtime check generation.<br><br>I agree.<br><br>> LAA also analyses & processes a whole loop whereas DA only checks dependences between 2 memory accesses, as well as decides whether it is profitable to generate runtime checks.<br><br>It processes innermost loops only and the fact that it can handle a whole loop rather than independent accesses I'm not sure it is a good path. For LAA's usage it's necessary but it creates a form of coupling (and complexity).<br><br>> 

There is definitely potential for improving the structure & organization of LAA, as well as improving the documentation. Happy to collaborate on that. <br><br>Are we really sure of that? Personally, I was thinking of submitting a patch but I'm not sure it is worth the effort. However, I'm glad to hear that you're happy to collaborate. :)<br>We can talk about that more if you want.<div><br></div><div>> I am not convinced it makes sense to add runtime check generating to DA directly, because I don’t think the static dependence checks really need to be strongly coupled with runtime-check generati<br><br>I agree to the latter, maybe to the former. In any case, I'd like to see the current DA staying as it is. And move the discussion to "what is the future of run-time checks". Either that</div><div>is extending LAA, DA or something else completely.<br><br>> To clarify, LAA does static checks and only generate runtime checks if it cannot prove that the dependence is safe for vectorization statically. Granted, the static checks mostly boil down to distance computations on SCEV expressions, but for the current use cases it seems to work well enough.<br><br>Yes, sorry for not stressing that LAA does static checks too as I said though, in my understanding they're very weak, though still enough for its usage. And I agree that LAA's capabilities is probably enough for innermost loop vectorization.<br>The important thing I believe is the future.<br><br>> It might be feasible to use DA for the static checks in LAA. That might help for a few multi-dimensional cases, but in practice generating proper runtime-checks for multi-dimensional cases is probably more important, due to aliasing issues.<br><br>Well... I tried that and it doesn't seem to be very useful unfortunately. The C/C++ way that arrays are defined is probably why DA is not that useful. Namely that a row can alias with another row in 2D arrays. The theory behind DA</div><div>is quite powerful if we knew that they don't alias. Right now, it just gives up.</div><div><br></div><div>I don't think that LAA can handle multi-dimensional cases either though, nor do I have a good idea about how to do it myself (in or out of LAA / DA).<br><br>Best,<br>Stefanos</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Στις Τετ, 8 Ιουλ 2020 στις 12:48 π.μ., ο/η Florian Hahn <<a href="mailto:florian_hahn@apple.com">florian_hahn@apple.com</a>> έγραψε:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi,<br>
<br>
> On Jul 7, 2020, at 18:37, Stefanos Baziotis via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> wrote:<br>
> <br>
> >  Ah, that's important information I didn't have.  Thank you!<br>
> <br>
> No problem, glad to help! <br>
> <br>
> To the rest of your thoughts, I certainly agree. One interesting question is why LAA<br>
> didn't use DA at all. Other than that, note that LAA is quite specialized, namely for<br>
> loop vectorization. Actually, it's even more specific. For innermost loop vectorization.<br>
> That affects the design. It might had been easier to create this specialized tool than<br>
> extending a general one (if that was a good path to follow is another topic).<br>
> <br>
> > But yet they are intimately related in that the kind of information you<br>
> > want to know statically and dynamically is the same.  I wonder what it<br>
> > would take to extend DA to generate runtime checks if it can't prove<br>
> > independence.<br>
> <br>
> Indeed, but again, IMHO unifying them is neither easy nor does it make sense.<br>
> They do fundamentally the same thing but their directions are very different.<br>
> <br>
> So, I see two options:<br>
> <br>
> a) As you said<br>
> <br>
> >  I wonder what it would take to extend DA to generate runtime checks if it can't prove independence.<br>
> <br>
> Personally, I see potential but neither do I know what it would take. Since this is something that I'm<br>
> currently thinking of, I would be more than interested to discuss it extensively.<br>
> <br>
> In any case, I would strongly prefer that we don't follow the LAA path, since I don't think it has potential<br>
> anyway. I think that we should try to find a way to extend it that is also based on strong theoretical foundation<br>
> and maintains the high quality of code.<br>
<br>
I am not sure if that is an entirely fair characterization of LAA. LAA is being used by the vectorizer (and other passes) in production for a few years now. None of the in-tree users of DA seem to be enabled by default and therefore LAA probably has an order of magnitude more testing, bug fixes & tuning.<br>
<br>
DA’s implementation might be cleaner, but as mentioned earlier, DA handles only a small subset of things LAA handles and hence I am not sure comparing the code-complexity is too helpful. <br>
IMO a lot of LAA complexity comes from things DA does not handle, in particular runtime check generation. LAA also analyses & processes a whole loop whereas DA only checks dependences between 2 memory accesses, as well as decides whether it is profitable to generate runtime checks.<br>
<br>
There is definitely potential for improving the structure & organization of LAA, as well as improving the documentation. Happy to collaborate on that. <br>
<br>
I am not convinced it makes sense to add runtime check generating to DA directly, because I don’t think the static dependence checks really need to be strongly coupled with runtime-check generation.<br>
<br>
> b) Extend LAA to do static checks<br>
> <br>
> The question here is though: Why do that? As I said, it doesn't seem to have potential and I believe that people<br>
> working on vectorizers (either LLVM's current one or external like e.g. RV and VPlan) don't do either.<br>
> <br>
<br>
To clarify, LAA does static checks and only generate runtime checks if it cannot prove that the dependence is safe for vectorization statically. Granted, the static checks mostly boil down to distance computations on SCEV expressions, but for the current use cases it seems to work well enough.<br>
<br>
It might be feasible to use DA for the static checks in LAA. That might help for a few multi-dimensional cases, but in practice generating proper runtime-checks for multi-dimensional cases is probably more important, due to aliasing issues.<br>
<br>
Cheers,<br>
Florian</blockquote></div>