From 3c2e365a672538b6aeee1a71ae9620f0ec7cb8a1 Mon Sep 17 00:00:00 2001 From: "L. Sousa" Date: Fri, 28 Aug 2026 18:33:01 +0100 Subject: [PATCH] refactor: remove intermediate-file cleanup option --- .../pt/up/fe/specs/clang/dumper/ClangAstDumper.java | 10 ---------- .../src/pt/up/fe/specs/clava/weaver/CxxWeaver.java | 8 -------- .../fe/specs/clava/weaver/options/CxxWeaverOption.java | 5 +---- .../specs/clava/weaver/options/CxxWeaverOptions.java | 2 -- 4 files changed, 1 insertion(+), 24 deletions(-) diff --git a/ClangAstParser/src/pt/up/fe/specs/clang/dumper/ClangAstDumper.java b/ClangAstParser/src/pt/up/fe/specs/clang/dumper/ClangAstDumper.java index b0c1463f0..de800f155 100644 --- a/ClangAstParser/src/pt/up/fe/specs/clang/dumper/ClangAstDumper.java +++ b/ClangAstParser/src/pt/up/fe/specs/clang/dumper/ClangAstDumper.java @@ -62,16 +62,6 @@ public static boolean usePlugin() { private final static String CLANG_DUMP_FILENAME = "clangDump.txt"; private final static String STDERR_DUMP_FILENAME = "stderr.txt"; - private static final List CLANG_AST_DUMPER_TEMP_FILES = List.of("includes.txt", CLANG_DUMP_FILENAME, - // "clavaDump.txt", "nodetypes.txt", "types.txt", "is_temporary.txt", "template_args.txt", - "clavaDump.txt", "nodetypes.txt", "types.txt", "is_temporary.txt", - "omp.txt", "invalid_source.txt", "enum_integer_type.txt", "consumer_order.txt", - "types_with_templates.txt"); - - public static List getTempFiles() { - return CLANG_AST_DUMPER_TEMP_FILES; - } - /** * TODO: Not implemented yet *

diff --git a/ClavaWeaver/src/pt/up/fe/specs/clava/weaver/CxxWeaver.java b/ClavaWeaver/src/pt/up/fe/specs/clava/weaver/CxxWeaver.java index ee675b71f..e76857769 100644 --- a/ClavaWeaver/src/pt/up/fe/specs/clava/weaver/CxxWeaver.java +++ b/ClavaWeaver/src/pt/up/fe/specs/clava/weaver/CxxWeaver.java @@ -12,7 +12,6 @@ import pt.up.fe.specs.clang.SupportedPlatform; import pt.up.fe.specs.clang.codeparser.CodeParser; import pt.up.fe.specs.clang.codeparser.ParallelCodeParser; -import pt.up.fe.specs.clang.dumper.ClangAstDumper; import pt.up.fe.specs.clava.ClavaLog; import pt.up.fe.specs.clava.ClavaNode; import pt.up.fe.specs.clava.ClavaOptions; @@ -797,13 +796,6 @@ protected boolean close() { SpecsIo.deleteFolder(new File(TEMP_SRC_FOLDER)); if (this.dataStore != null) { - // Delete intermediary files - if (this.dataStore.get(CxxWeaverOption.CLEAN_INTERMEDIATE_FILES)) { - for (String tempFile : ClangAstDumper.getTempFiles()) { - new File(tempFile).delete(); - } - } - // Re-enable output if (this.dataStore.get(CxxWeaverOption.DISABLE_CLAVA_INFO)) { SpecsLogs.getSpecsLogger().setLevelAll(null); diff --git a/ClavaWeaver/src/pt/up/fe/specs/clava/weaver/options/CxxWeaverOption.java b/ClavaWeaver/src/pt/up/fe/specs/clava/weaver/options/CxxWeaverOption.java index a86228dfd..c9074f6ab 100644 --- a/ClavaWeaver/src/pt/up/fe/specs/clava/weaver/options/CxxWeaverOption.java +++ b/ClavaWeaver/src/pt/up/fe/specs/clava/weaver/options/CxxWeaverOption.java @@ -37,9 +37,6 @@ public interface CxxWeaverOption { DataKey CHECK_SYNTAX = KeyFactory.bool("Check C/CXX Syntax") .setLabel("Check C/C++ syntax (performs additional parsing step)"); - DataKey CLEAN_INTERMEDIATE_FILES = KeyFactory.bool("Clean intermediate files") - .setDefault(() -> true); - DataKey HEADER_INCLUDES = LaraIKeyFactory.folderList("header includes") .setLabel("Normal Includes") .setDefault(FileList::newInstance); @@ -85,7 +82,7 @@ public interface CxxWeaverOption { StoreDefinition STORE_DEFINITION = new StoreDefinitionBuilder("C/C++ Weaver") .addKeys(ClavaOptions.STORE_DEFINITION.getKeys()) - .addKeys(WOVEN_CODE_FOLDERNAME, DISABLE_CLAVA_INFO, CHECK_SYNTAX, CLEAN_INTERMEDIATE_FILES, + .addKeys(WOVEN_CODE_FOLDERNAME, DISABLE_CLAVA_INFO, CHECK_SYNTAX, HEADER_INCLUDES, // SKIP_HEADER_INCLUDES_PARSING, PARSE_INCLUDES, diff --git a/ClavaWeaver/src/pt/up/fe/specs/clava/weaver/options/CxxWeaverOptions.java b/ClavaWeaver/src/pt/up/fe/specs/clava/weaver/options/CxxWeaverOptions.java index 639719e1e..814e517b2 100644 --- a/ClavaWeaver/src/pt/up/fe/specs/clava/weaver/options/CxxWeaverOptions.java +++ b/ClavaWeaver/src/pt/up/fe/specs/clava/weaver/options/CxxWeaverOptions.java @@ -63,8 +63,6 @@ public class CxxWeaverOptions { addBooleanOption(CxxWeaverOption.CHECK_SYNTAX, "cs", "check-syntax", "Checks syntax of woven code"); - addBooleanOption(CxxWeaverOption.CLEAN_INTERMEDIATE_FILES, "cl", "clean", "Clean intermediate files"); - addBooleanOption(CxxWeaverOption.DISABLE_CODE_GENERATION, "ncg", "no-code-gen", "Disables automatic code generation");