Skip to content

Commit 3843d41

Browse files
igorban-inteligcbot
authored andcommitted
Always allow scalars for pre-llvm16
.
1 parent 31b3a5e commit 3843d41

File tree

2 files changed

+8
-1
lines changed

2 files changed

+8
-1
lines changed

IGC/VectorCompiler/lib/GenXCodeGen/GenXAddressCommoning.cpp

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1007,7 +1007,13 @@ bool GenXAddressCommoning::vectorizeAddrsFromOneVector(
10071007
auto ui = OldConv->use_begin();
10081008
auto user = cast<Instruction>(ui->getUser());
10091009
auto NewExtract = R2.createRdRegion(NewConv, OldConv->getName(), user,
1010-
user->getDebugLoc(), /*ScalarAllowed=*/!OldConv->getType()->isVectorTy());
1010+
user->getDebugLoc(),
1011+
#if LLVM_VERSION_MAJOR < 16
1012+
/*ScalarAllowed=*/true);
1013+
#else
1014+
/*ScalarAllowed=*/
1015+
!OldConv->getType()->isVectorTy());
1016+
#endif
10111017
Numbering->setNumber(NewExtract, Numbering->getNumber(user));
10121018
// At this late stage, I believe nothing relies on the baling type for
10131019
// this instruction being set to RDREGION, but we set it anyway for

IGC/VectorCompiler/test/GenXAddressCommoning/rdregion_reduce.ll

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@
99
; RUN: %opt_opaque_ptrs %use_old_pass_manager% -GenXModule -GenXLiveRangesWrapper \
1010
; RUN: -GenXAddressCommoningWrapper -march=genx64 -mtriple=spir64-unknown-unknown \
1111
; RUN: -mcpu=Gen9 -S < %s | FileCheck %s
12+
; REQUIRES: llvm_16_or_greater
1213

1314
target datalayout = "e-p:64:64-p3:32:32-p6:32:32-i64:64-n8:16:32:64"
1415
target triple = "spir64-unknown-unknown"

0 commit comments

Comments
 (0)