[PATCH] D152441: [SLPVectorizer] Don't include debug instructions in ScheduleRegionSize

Alexey Bataev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 8 07:51:53 PDT 2023


ABataev added inline comments.


================
Comment at: llvm/test/Transforms/SLPVectorizer/X86/schedule_budget.ll:8
 ; Test if the budget for the scheduling region size works.
 ; We test with a reduced budget of 16 which should prevent vectorizing the loads.
 
----------------
bjope wrote:
> So this test case is kind of "broken" already on trunk?
> 
> (Looks like commit 352c46e70716061e99cae2009daddbfc78380fda changed this test in a way so that the first loads are vectorized even with a budget at 16.)
Not broken, just changed. You'd better to add a new test with budget limits and debug info.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D152441



More information about the llvm-commits mailing list