From 9cf27e323f19e73aa4a8e9bd1f3df57efe61f243 Mon Sep 17 00:00:00 2001 From: "Tung D. Le" Date: Wed, 15 Jan 2025 02:03:26 -0500 Subject: [PATCH] Fix errors in Release mode Signed-off-by: Tung D. Le --- src/Accelerators/NNPA/Compiler/NNPACompilerOptions.cpp | 9 ++++----- .../NNPA/Conversion/ONNXToZHigh/ONNXToZHighCommon.cpp | 2 +- src/Accelerators/NNPA/Pass/NNPAPasses.hpp | 2 +- 3 files changed, 6 insertions(+), 7 deletions(-) diff --git a/src/Accelerators/NNPA/Compiler/NNPACompilerOptions.cpp b/src/Accelerators/NNPA/Compiler/NNPACompilerOptions.cpp index 0c28a7a79f..3312eb396b 100644 --- a/src/Accelerators/NNPA/Compiler/NNPACompilerOptions.cpp +++ b/src/Accelerators/NNPA/Compiler/NNPACompilerOptions.cpp @@ -132,11 +132,10 @@ llvm::cl::list> nnpaQuantOpTypesOpt( "nnpa-quant-op-types", llvm::cl::desc( "A comma-separated list of types of operations that are quantized. " - "E.g. 'MatMul,Conv'. Strings for types are the same as ONNX " - "operator " - "names in https://onnx.ai/onnx/operators/. By default or with " - "specifying this option, the compiler will determine the operation " - "types by itself."), + "E.g. 'MatMul,Conv'. Strings for types are the same as ONNX operator " + "names in https://onnx.ai/onnx/operators/. Currently, only MatMul is " + "supported. Without specifying this option, the compiler will " + "determine the operation types by itself."), llvm::cl::location(nnpaQuantOpTypes), llvm::cl::ValueOptional, llvm::cl::CommaSeparated, llvm::cl::cat(OnnxMlirCommonOptions)); diff --git a/src/Accelerators/NNPA/Conversion/ONNXToZHigh/ONNXToZHighCommon.cpp b/src/Accelerators/NNPA/Conversion/ONNXToZHigh/ONNXToZHighCommon.cpp index ce69ea1658..fcceae76aa 100644 --- a/src/Accelerators/NNPA/Conversion/ONNXToZHigh/ONNXToZHighCommon.cpp +++ b/src/Accelerators/NNPA/Conversion/ONNXToZHigh/ONNXToZHighCommon.cpp @@ -114,7 +114,7 @@ std::vector ONNXToZHighLoweringConfiguration::Quant::opTypes = {}; void configureONNXToZHighLoweringPass(bool optReportNNPAUnsupportedOps, bool isDynQuant, bool quantIsActivationSym, bool quantIsWeightSym, - ArrayRef quantOpTypes) { + std::vector quantOpTypes) { ONNXToZHighLoweringConfiguration::optReportNNPAUnsupportedOps = optReportNNPAUnsupportedOps; ONNXToZHighLoweringConfiguration::isDynQuant = isDynQuant; diff --git a/src/Accelerators/NNPA/Pass/NNPAPasses.hpp b/src/Accelerators/NNPA/Pass/NNPAPasses.hpp index c23fb7f158..92d9691595 100644 --- a/src/Accelerators/NNPA/Pass/NNPAPasses.hpp +++ b/src/Accelerators/NNPA/Pass/NNPAPasses.hpp @@ -32,7 +32,7 @@ std::unique_ptr createDevicePlacementPass( std::unique_ptr createONNXToZHighPass(); void configureONNXToZHighLoweringPass(bool reportOnNNPAUnsupportedOps, bool isDynQuant, bool quantIsActivationSym, bool quantIsWeightSym, - llvm::ArrayRef quantOpTypes); + std::vector quantOpTypes); /// Add pass for rewriting ONNX ops for ZHigh. std::unique_ptr createRewriteONNXForZHighPass();