shylie/llvm-project
1
0
Fork 0
Code Issues Pull Requests Actions 6 Packages Projects Releases Wiki Activity

[NFC][MLIR][OpenMP] Correct attach_none to attach_never (#176855)

Browse Source 
Originally gave attach_never the incorrect name, so this patch corrects
that to keep things consistent everywhere.
This commit is contained in:
agozillon 2026-01-20 15:43:27 +01:00 committed by GitHub
parent 08391d2b4d
commit c16340e49e
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
3 changed files with 8 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
mlir/include/mlir/Dialect/OpenMP/OpenMPEnums.td
View File

@ -123,7 +123,7 @@ def ClauseMapFlagsPresent : I32BitEnumAttrCaseBit<"present", 10>;
def ClauseMapFlagsOMPXHold : I32BitEnumAttrCaseBit<"ompx_hold", 11>;
def ClauseMapFlagsAttach : I32BitEnumAttrCaseBit<"attach", 12>;
def ClauseMapFlagsAttachAlways : I32BitEnumAttrCaseBit<"attach_always", 13>;
def ClauseMapFlagsAttachNone : I32BitEnumAttrCaseBit<"attach_none", 14>;
def ClauseMapFlagsAttachNever : I32BitEnumAttrCaseBit<"attach_never", 14>;
def ClauseMapFlagsAttachAuto : I32BitEnumAttrCaseBit<"attach_auto", 15>;
def ClauseMapFlagsRefPtr : I32BitEnumAttrCaseBit<"ref_ptr", 16>;
def ClauseMapFlagsRefPtee : I32BitEnumAttrCaseBit<"ref_ptee", 17>;
@ -148,7 +148,7 @@ def ClauseMapFlags : OpenMP_BitEnumAttr<
ClauseMapFlagsOMPXHold,
ClauseMapFlagsAttach,
ClauseMapFlagsAttachAlways,
ClauseMapFlagsAttachNone,
ClauseMapFlagsAttachNever,
ClauseMapFlagsAttachAuto,
ClauseMapFlagsRefPtr,
ClauseMapFlagsRefPtee,

8
mlir/lib/Dialect/OpenMP/IR/OpenMPDialect.cpp
View File

@ -1803,8 +1803,8 @@ static ParseResult parseMapClause(OpAsmParser &parser,
if (mapTypeMod == "attach_always")
mapTypeBits |= ClauseMapFlags::attach_always;
if (mapTypeMod == "attach_none")
mapTypeBits |= ClauseMapFlags::attach_none;
if (mapTypeMod == "attach_never")
mapTypeBits |= ClauseMapFlags::attach_never;
if (mapTypeMod == "attach_auto")
mapTypeBits |= ClauseMapFlags::attach_auto;
@ -1880,8 +1880,8 @@ static void printMapClause(OpAsmPrinter &p, Operation *op,
mapTypeStrs.push_back("attach");
if (mapTypeToBool(mapFlags, ClauseMapFlags::attach_always))
mapTypeStrs.push_back("attach_always");
if (mapTypeToBool(mapFlags, ClauseMapFlags::attach_none))
mapTypeStrs.push_back("attach_none");
if (mapTypeToBool(mapFlags, ClauseMapFlags::attach_never))
mapTypeStrs.push_back("attach_never");
if (mapTypeToBool(mapFlags, ClauseMapFlags::attach_auto))
mapTypeStrs.push_back("attach_auto");
if (mapTypeToBool(mapFlags, ClauseMapFlags::ref_ptr))

4
mlir/test/Dialect/OpenMP/ops.mlir
View File

@ -3340,7 +3340,7 @@ func.func @omp_target_map_clause_type_test(%arg0 : memref<?xi32>) -> () {
// CHECK: %{{.*}}map_clauses(ompx_hold){{.*}}
// CHECK: %{{.*}}map_clauses(attach){{.*}}
// CHECK: %{{.*}}map_clauses(attach_always){{.*}}
// CHECK: %{{.*}}map_clauses(attach_none){{.*}}
// CHECK: %{{.*}}map_clauses(attach_never){{.*}}
// CHECK: %{{.*}}map_clauses(attach_auto){{.*}}
// CHECK: %{{.*}}map_clauses(ref_ptr){{.*}}
// CHECK: %{{.*}}map_clauses(ref_ptee){{.*}}
@ -3360,7 +3360,7 @@ func.func @omp_target_map_clause_type_test(%arg0 : memref<?xi32>) -> () {
%mapv12 = omp.map.info var_ptr(%arg0 : memref<?xi32>, tensor<?xi32>) map_clauses(ompx_hold) capture(ByRef) -> memref<?xi32> {name = ""}
%mapv13 = omp.map.info var_ptr(%arg0 : memref<?xi32>, tensor<?xi32>) map_clauses(attach) capture(ByRef) -> memref<?xi32> {name = ""}
%mapv14 = omp.map.info var_ptr(%arg0 : memref<?xi32>, tensor<?xi32>) map_clauses(attach_always) capture(ByRef) -> memref<?xi32> {name = ""}
%mapv15 = omp.map.info var_ptr(%arg0 : memref<?xi32>, tensor<?xi32>) map_clauses(attach_none) capture(ByRef) -> memref<?xi32> {name = ""}
%mapv15 = omp.map.info var_ptr(%arg0 : memref<?xi32>, tensor<?xi32>) map_clauses(attach_never) capture(ByRef) -> memref<?xi32> {name = ""}
%mapv16 = omp.map.info var_ptr(%arg0 : memref<?xi32>, tensor<?xi32>) map_clauses(attach_auto) capture(ByRef) -> memref<?xi32> {name = ""}
%mapv17 = omp.map.info var_ptr(%arg0 : memref<?xi32>, tensor<?xi32>) map_clauses(ref_ptr) capture(ByRef) -> memref<?xi32> {name = ""}
%mapv18 = omp.map.info var_ptr(%arg0 : memref<?xi32>, tensor<?xi32>) map_clauses(ref_ptee) capture(ByRef) -> memref<?xi32> {name = ""}