search for: d15185

Thank you both for the inputs. I've created a patch for the same, please review: http://reviews.llvm.org/D15185 >You can specifically insert it into the pass pipeline, but in this case, we should just fix SLP to preserve the analysis. @Hal, I tried doing this but it didn't seem to work. I added the GlobalsAA pass right before GVN but it didn't seem to help. In retrospect, looking at the way pass...

...ehdi Amini <mehdi.amini at apple.com> wrote: > > On Dec 3, 2015, at 4:21 AM, Vaivaswatha Nagaraj via llvm-dev < > llvm-dev at lists.llvm.org> wrote: > > Thank you both for the inputs. I've created a patch for the same, please > review: > http://reviews.llvm.org/D15185 > > >You can specifically insert it into the pass pipeline, but in this case, > we should just fix SLP to preserve the analysis. > @Hal, I tried doing this but it didn't seem to work. I added the GlobalsAA > pass right before GVN but it didn't seem to help. In retrospect,...

...ple.com> wrote: > >> >> On Dec 3, 2015, at 4:21 AM, Vaivaswatha Nagaraj via llvm-dev < >> llvm-dev at lists.llvm.org> wrote: >> >> Thank you both for the inputs. I've created a patch for the same, please >> review: >> http://reviews.llvm.org/D15185 >> >> >You can specifically insert it into the pass pipeline, but in this >> case, we should just fix SLP to preserve the analysis. >> @Hal, I tried doing this but it didn't seem to work. I added the >> GlobalsAA pass right before GVN but it didn't seem to h...

Hi James, Thanks for the help. From the log, I could infer that SLP vectorizer is not preserving alias analysis, preventing GVN from getting the info. Although the first function to get compiled has GlobalsAA available during GVN, rest of them do not as SLP vectorizer run on that function invalidates GlobalsAA which is a module pass. Is there a way to force re-computation of a particular