Skip to content

Commit 7ab3f55

Browse files
author
Rubén
authored
Merge pull request compiler-explorer#937 from kalray/pr-gccdump
Always try to attach gccdump info
2 parents 7c8bb70 + 13f6e80 commit 7ab3f55

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

lib/base-compiler.js

+5-4
Original file line numberDiff line numberDiff line change
@@ -296,6 +296,11 @@ class BaseCompiler {
296296
.then(([asmResult, astResult, gccDumpResult]) => {
297297
asmResult.dirPath = dirPath;
298298
asmResult.compilationOptions = options;
299+
// Here before the check to ensure dump reports even on failure cases
300+
if (this.compiler.supportsGccDump && gccDumpResult) {
301+
asmResult.gccDumpOutput = gccDumpResult;
302+
}
303+
299304
if (asmResult.code !== 0) {
300305
asmResult.asm = "<Compilation failed>";
301306
return asmResult;
@@ -313,10 +318,6 @@ class BaseCompiler {
313318
asmResult.astOutput = astResult;
314319
}
315320

316-
if (this.compiler.supportsGccDump && gccDumpResult) {
317-
asmResult.gccDumpOutput = gccDumpResult;
318-
}
319-
320321
return this.checkOutputFileAndDoPostProcess(asmResult, outputFilename, filters);
321322
});
322323
});

0 commit comments

Comments
 (0)