diff options
| author | RabsRincon <ruben@rinconblanco.es> | 2022-10-12 20:44:38 +0200 |
|---|---|---|
| committer | RabsRincon <ruben@rinconblanco.es> | 2022-10-12 20:44:38 +0200 |
| commit | 6ce4898bfe939a374b28d60dcca02bf6192a52e5 (patch) | |
| tree | 3475bcf80a4179508d0b585122a51e0c15b025c0 | |
| parent | 70754d0b15ca532a5b650581b8ed21fcaf51dc7a (diff) | |
| download | compiler-explorer-gh-4503.tar.gz compiler-explorer-gh-4503.zip | |
Properly allow for multiple opt pass panesgh-4503
| -rw-r--r-- | static/panes/compiler.js | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/static/panes/compiler.js b/static/panes/compiler.js index 459e9aa6b..b38cb7da0 100644 --- a/static/panes/compiler.js +++ b/static/panes/compiler.js @@ -1672,7 +1672,6 @@ Compiler.prototype.onIrViewClosed = function (id) { Compiler.prototype.onLLVMOptPipelineViewOpened = function (id) { if (this.id === id) { - this.llvmOptPipelineButton.prop('disabled', true); this.llvmOptPipelineViewOpen = true; this.compile(); } @@ -1680,7 +1679,6 @@ Compiler.prototype.onLLVMOptPipelineViewOpened = function (id) { Compiler.prototype.onLLVMOptPipelineViewClosed = function (id) { if (this.id === id) { - this.llvmOptPipelineButton.prop('disabled', false); this.llvmOptPipelineViewOpen = false; } }; @@ -2237,7 +2235,8 @@ Compiler.prototype.updateButtons = function () { this.ppButton.prop('disabled', this.ppViewOpen); this.astButton.prop('disabled', this.astViewOpen); this.irButton.prop('disabled', this.irViewOpen); - this.llvmOptPipelineButton.prop('disabled', this.llvmOptPipelineViewOpen); + // As per #4112, it's useful to have this available more than once: Don't disable it when it opens + // this.llvmOptPipelineButton.prop('disabled', this.llvmOptPipelineViewOpen); this.deviceButton.prop('disabled', this.deviceViewOpen); this.rustMirButton.prop('disabled', this.rustMirViewOpen); this.haskellCoreButton.prop('disabled', this.haskellCoreViewOpen); @@ -2256,8 +2255,8 @@ Compiler.prototype.updateButtons = function () { this.ppButton.toggle(!!this.compiler.supportsPpView); this.astButton.toggle(!!this.compiler.supportsAstView); this.irButton.toggle(!!this.compiler.supportsIrView); - // As per #4112, it's useful to have this available more than once: Don't disable it when it opens - //this.llvmOptPipelineButton.toggle(!!this.compiler.supportsLLVMOptPipelineView); + + this.llvmOptPipelineButton.toggle(!!this.compiler.supportsLLVMOptPipelineView); this.deviceButton.toggle(!!this.compiler.supportsDeviceAsmView); this.rustMirButton.toggle(!!this.compiler.supportsRustMirView); this.rustMacroExpButton.toggle(!!this.compiler.supportsRustMacroExpView); |