From bb90520632ee50ce2c69939caca79a3eabbb4a66 Mon Sep 17 00:00:00 2001 From: Scott Manley Date: Tue, 18 Nov 2025 11:46:02 -0800 Subject: [PATCH] [OpenACC] add cl::values to ACCImplicitRoutineOptions Add the cl::values to the pass options so an assert is not reached when trying to generate a reproducer e.g. "unknown data value for option" --- mlir/include/mlir/Dialect/OpenACC/Transforms/Passes.td | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/mlir/include/mlir/Dialect/OpenACC/Transforms/Passes.td b/mlir/include/mlir/Dialect/OpenACC/Transforms/Passes.td index 970d9304d8289..cad78df2fbb0b 100644 --- a/mlir/include/mlir/Dialect/OpenACC/Transforms/Passes.td +++ b/mlir/include/mlir/Dialect/OpenACC/Transforms/Passes.td @@ -97,7 +97,14 @@ def ACCImplicitRoutine : Pass<"acc-implicit-routine", "mlir::ModuleOp"> { "mlir::acc::DeviceType::None", "Target device type for implicit routine generation. " "Ensures that `acc routine` device_type clauses are " - "properly considered not just default clauses."> + "properly considered not just default clauses.", + [{::llvm::cl::values( + clEnumValN(mlir::acc::DeviceType::None, "none", "none"), + clEnumValN(mlir::acc::DeviceType::Host, "host", "host"), + clEnumValN(mlir::acc::DeviceType::Multicore, "multicore", "multicore"), + clEnumValN(mlir::acc::DeviceType::Nvidia, "nvidia", "nvidia"), + clEnumValN(mlir::acc::DeviceType::Radeon, "radeon", "radeon")) + }]> ]; }