Skip to content

Commit 863abed

Browse files
committed
Fix sycl-post-link emitting warning incorrectly
1 parent 3082834 commit 863abed

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

llvm/tools/sycl-post-link/sycl-post-link.cpp

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -799,11 +799,10 @@ processInputModule(std::unique_ptr<Module> M) {
799799
std::unique_ptr<module_split::ModuleSplitterBase> ESIMDSplitter =
800800
module_split::getSplitterByKernelType(std::move(MDesc),
801801
EmitOnlyKernelsAsEntryPoints);
802-
SplitOccurred |= ESIMDSplitter->remainingSplits() > 1;
803-
Modified |= SplitOccurred;
802+
bool ESIMDSplitOccurred = ESIMDSplitter->remainingSplits() > 1;
804803

805-
if (SplitOccurred && (SplitMode == module_split::SPLIT_PER_KERNEL) &&
806-
!SplitEsimd) {
804+
if (ESIMDSplitOccurred && SplitOccurred &&
805+
(SplitMode == module_split::SPLIT_PER_KERNEL) && !SplitEsimd) {
807806
// Controversial state reached - SYCL and ESIMD entry points resulting
808807
// from SYCL/ESIMD split (which is done always) are linked back, since
809808
// -split-esimd is not specified, but per-kernel split is requested.
@@ -812,6 +811,8 @@ processInputModule(std::unique_ptr<Module> M) {
812811
SplitEsimd.ValueStr + " must also be specified");
813812
}
814813
SmallVector<module_split::ModuleDesc, 2> MMs;
814+
SplitOccurred |= ESIMDSplitOccurred;
815+
Modified |= SplitOccurred;
815816

816817
while (ESIMDSplitter->hasMoreSplits()) {
817818
module_split::ModuleDesc MDesc2 = ESIMDSplitter->nextSplit();

0 commit comments

Comments
 (0)