Skip to content

Commit acff3cc

Browse files
gpu - swap out bitwise assignment operators for bools
Co-authored-by: Zach Atkins <[email protected]>
1 parent 8d12f40 commit acff3cc

File tree

2 files changed

+12
-12
lines changed

2 files changed

+12
-12
lines changed

backends/cuda-gen/ceed-cuda-gen-operator-build.cpp

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -954,12 +954,12 @@ extern "C" int CeedOperatorBuildKernel_Cuda_gen(CeedOperator op, bool *is_good_b
954954
Ceed basis_ceed;
955955

956956
CeedCallBackend(CeedBasisIsTensor(basis, &is_tensor));
957-
is_all_tensor &= is_tensor;
958-
is_all_nontensor &= !is_tensor;
957+
is_all_tensor = is_all_tensor && is_tensor;
958+
is_all_nontensor = is_all_not_tensor && !is_tensor;
959959
CeedCallBackend(CeedBasisGetCeed(basis, &basis_ceed));
960960
CeedCallBackend(CeedGetResource(basis_ceed, &resource));
961961
CeedCallBackend(CeedGetResourceRoot(basis_ceed, resource, ":", &resource_root));
962-
has_shared_bases &= !strcmp(resource_root, "/gpu/cuda/shared");
962+
has_shared_bases = has_shared_bases && !strcmp(resource_root, "/gpu/cuda/shared");
963963
CeedCallBackend(CeedFree(&resource_root));
964964
CeedCallBackend(CeedDestroy(&basis_ceed));
965965
}
@@ -977,13 +977,13 @@ extern "C" int CeedOperatorBuildKernel_Cuda_gen(CeedOperator op, bool *is_good_b
977977
Ceed basis_ceed;
978978

979979
CeedCallBackend(CeedBasisIsTensor(basis, &is_tensor));
980-
is_all_tensor &= is_tensor;
981-
is_all_nontensor &= !is_tensor;
980+
is_all_tensor = is_all_tensor && is_tensor;
981+
is_all_nontensor = is_all_nontensor && !is_tensor;
982982

983983
CeedCallBackend(CeedBasisGetCeed(basis, &basis_ceed));
984984
CeedCallBackend(CeedGetResource(basis_ceed, &resource));
985985
CeedCallBackend(CeedGetResourceRoot(basis_ceed, resource, ":", &resource_root));
986-
has_shared_bases &= !strcmp(resource_root, "/gpu/cuda/shared");
986+
has_shared_bases = has_shared_bases && !strcmp(resource_root, "/gpu/cuda/shared");
987987
CeedCallBackend(CeedFree(&resource_root));
988988
CeedCallBackend(CeedDestroy(&basis_ceed));
989989
}

backends/hip-gen/ceed-hip-gen-operator-build.cpp

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -980,12 +980,12 @@ extern "C" int CeedOperatorBuildKernel_Hip_gen(CeedOperator op, bool *is_good_bu
980980
Ceed basis_ceed;
981981

982982
CeedCallBackend(CeedBasisIsTensor(basis, &is_tensor));
983-
is_all_tensor &= is_tensor;
984-
is_all_nontensor &= !is_tensor;
983+
is_all_tensor = is_all_tensor && is_tensor;
984+
is_all_nontensor = is_all_nontensor && !is_tensor;
985985
CeedCallBackend(CeedBasisGetCeed(basis, &basis_ceed));
986986
CeedCallBackend(CeedGetResource(basis_ceed, &resource));
987987
CeedCallBackend(CeedGetResourceRoot(basis_ceed, resource, ":", &resource_root));
988-
has_shared_bases &= !strcmp(resource_root, "/gpu/hip/shared");
988+
has_shared_bases = has_shared_bases && !strcmp(resource_root, "/gpu/hip/shared");
989989
CeedCallBackend(CeedFree(&resource_root));
990990
CeedCallBackend(CeedDestroy(&basis_ceed));
991991
}
@@ -1003,13 +1003,13 @@ extern "C" int CeedOperatorBuildKernel_Hip_gen(CeedOperator op, bool *is_good_bu
10031003
Ceed basis_ceed;
10041004

10051005
CeedCallBackend(CeedBasisIsTensor(basis, &is_tensor));
1006-
is_all_tensor &= is_tensor;
1007-
is_all_nontensor &= !is_tensor;
1006+
is_all_tensor = is_all_tensor && is_tensor;
1007+
is_all_nontensor = is_all_nontensor && !is_tensor;
10081008

10091009
CeedCallBackend(CeedBasisGetCeed(basis, &basis_ceed));
10101010
CeedCallBackend(CeedGetResource(basis_ceed, &resource));
10111011
CeedCallBackend(CeedGetResourceRoot(basis_ceed, resource, ":", &resource_root));
1012-
has_shared_bases &= !strcmp(resource_root, "/gpu/hip/shared");
1012+
has_shared_bases = has_shared_bases && !strcmp(resource_root, "/gpu/hip/shared");
10131013
CeedCallBackend(CeedFree(&resource_root));
10141014
CeedCallBackend(CeedDestroy(&basis_ceed));
10151015
}

0 commit comments

Comments
 (0)