[PATCH] D106653: [LoopVectorize][AArch64] Enable ordered reductions by default for AArch64
Sanjay Patel via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Aug 6 05:37:24 PDT 2021
spatel added inline comments.
================
Comment at: llvm/test/Transforms/LoopVectorize/AArch64/scalable-strict-fadd.ll:5
; RUN: opt < %s -loop-vectorize -scalable-vectorization=on -mtriple aarch64-unknown-linux-gnu -mattr=+sve -force-ordered-reductions=true -hints-allow-reordering=true -S 2>%t | FileCheck %s --check-prefix=CHECK-UNORDERED
+; RUN: opt < %s -loop-vectorize -scalable-vectorization=on -mtriple aarch64-unknown-linux-gnu -mattr=+sve -hints-allow-reordering=false -S 2>%t | FileCheck %s --check-prefix=CHECK-ORDERED
----------------
Drive-by comment (checking my understanding) - we could add this RUN line with `--check-prefix=CHECK-NOT-VECTORIZED` as a preliminary commit and then update only the prefix to show the change in behavior with this patch?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D106653/new/
https://reviews.llvm.org/D106653
More information about the llvm-commits
mailing list