From cf784c71f5b278d77dcf99a076445c76f1509c0b Mon Sep 17 00:00:00 2001 From: mshahid Date: Mon, 17 Feb 2025 02:58:28 -0800 Subject: [PATCH] -Simplifies the target feature initialization option. Adds a Todo comment to plugin the target platform intialization using target descriptor. --- lib/TPP/DefaultPipeline.cpp | 2 +- tools/tpp-run/tpp-run.cpp | 1 + 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/lib/TPP/DefaultPipeline.cpp b/lib/TPP/DefaultPipeline.cpp index cdf84a79b..45488eb30 100644 --- a/lib/TPP/DefaultPipeline.cpp +++ b/lib/TPP/DefaultPipeline.cpp @@ -189,7 +189,7 @@ struct DefaultPipeline : public tpp::impl::DefaultPipelineBase, // Lower to LLVM ConvertVectorToLLVMPassOptions options; - options.amx = vnni::utils::hasAMX() ? true : false; + options.amx = vnni::utils::hasAMX(); pm.addPass(createConvertVectorToLLVMPass(options)); pm.addPass(createFinalizeMemRefToLLVMConversionPass()); pm.addPass(createConvertSCFToCFPass()); diff --git a/tools/tpp-run/tpp-run.cpp b/tools/tpp-run/tpp-run.cpp index 6ccf3d909..7a2a7d7f1 100644 --- a/tools/tpp-run/tpp-run.cpp +++ b/tools/tpp-run/tpp-run.cpp @@ -272,6 +272,7 @@ int main(int argc, char **argv) { return 1; // Initialize the underlying platform + // TODO: Move this to use the target information flags libxsmm_init(); // Initialize the LLVM machinery llvm::InitLLVM y(argc, argv);