[PATCH] D134605: [SLP] Add ScalarizationOverheadBuilder helper to track vector extractions

Alan Zhao via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 28 09:57:52 PDT 2022


ayzhao added a comment.

In D134605#3820913 <https://reviews.llvm.org/D134605#3820913>, @uabelho wrote:

> In D134605#3820876 <https://reviews.llvm.org/D134605#3820876>, @RKSimon wrote:
>
>> Thanks @uabelho my testing suggests rG926ccfef032d206dcbcdf74ca1e3a9ebf4d1be45 <https://reviews.llvm.org/rG926ccfef032d206dcbcdf74ca1e3a9ebf4d1be45> fixed the issue, I'll see whether the fix can be improved before I commit the regression test.
>
> Yes I've also verified that the crash disappears with the fix.

+1, I can also verify that the crash disappears on Chrome builds with this fix. Thanks!


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D134605/new/

https://reviews.llvm.org/D134605



More information about the llvm-commits mailing list