[PATCH] D157549: [LV] Add debug output to print interleaved groups

Anna Thomas via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 8 09:26:35 PDT 2023


anna added a comment.

shall I go ahead and land this? Simple enough change.



================
Comment at: llvm/lib/Analysis/VectorUtils.cpp:31
 
+const char VerboseDebug[] = DEBUG_TYPE "-verbose";
+
----------------
fhahn wrote:
> IMO it would be fine to print this with general debug?
Sure, it would update many tests, but that would be better I suppose to see what happens.


================
Comment at: llvm/test/Transforms/LoopVectorize/X86/vector_ptr_load_store.ll:1
-; RUN: opt -passes=loop-vectorize -mcpu=corei7-avx -debug -S < %s 2>&1 | FileCheck %s
+; RUN: opt -passes=loop-vectorize -mcpu=corei7-avx -debug-only=loop-vectorize -S < %s 2>&1 | FileCheck %s
 ; REQUIRES: asserts
----------------
fhahn wrote:
> unrelated change?
With the `-debug` flag it was printing out the interleaved groups as well, which I felt takes away focus from the main check lines. 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D157549



More information about the llvm-commits mailing list