[GR-15995] Do not add Graal at runtime, and instead require including it at build time.

PullRequest: truffleruby/857

Author: eregon

bench/optcarrot/README.md

@@ -9,7 +9,7 @@ The code is imported from revision `898783836dfcec04d25a8e7d4b1ce3ce3bbcebdd`.
 
 The emulator can be run with the SDL2 GUI with:
 ```
-$ jt install graal
+$ jt build --graal
 
-$ jt ruby --graal bench/optcarrot/bin/optcarrot --sdl2 --audio=none bench/optcarrot/examples/Lan_Master.nes
+$ jt ruby bench/optcarrot/bin/optcarrot --sdl2 --audio=none bench/optcarrot/examples/Lan_Master.nes
 ```

ci.jsonnet

@@ -136,14 +136,9 @@ local part_definitions = {
   graal: {
     core: {
       is_after+:: ["$.use.build"],
-      setup+: [
-        ["cd", "../graal/compiler"],
-        ["mx", "build"],
-        ["cd", "../../main"],
-      ],
+      setup+: jt(["build", "--graal"]),
 
       environment+: {
-        GRAAL_HOME: "../graal/compiler",
         HOST_VM: "server",
         HOST_VM_CONFIG: "graal-core",
       },
@@ -153,7 +148,6 @@ local part_definitions = {
       environment+: {
         HOST_VM: "server",
         HOST_VM_CONFIG: "default",
-        MX_NO_GRAAL: "true",
       },
     },
 
@@ -170,14 +164,9 @@ local part_definitions = {
         ["git", "-C", repo, "checkout", find_commit_importing_truffle],
       ],
 
-      setup+: self.clone + [
-        ["cd", "../graal-enterprise/graal-enterprise"],
-        ["mx", "build"],
-        ["cd", "../../main"],
-      ],
+      setup+: self.clone + jt(["build", "--dy", "/graal-enterprise"]),
 
       environment+: {
-        GRAAL_HOME: "$BUILD_DIR/graal-enterprise/graal-enterprise",
         HOST_VM: "server",
         HOST_VM_CONFIG: "graal-enterprise",
       },
@@ -201,7 +190,6 @@ local part_definitions = {
 
       environment+: {
         HOST_VM_CONFIG: "graal-core",
-        GRAAL_HOME: "$BUILD_DIR/graal/compiler",
         VM_SUITE_HOME: "$BUILD_DIR/graal/vm",
       },
     },
@@ -215,7 +203,6 @@ local part_definitions = {
 
       environment+: {
         HOST_VM_CONFIG: "graal-enterprise",
-        GRAAL_HOME: "$BUILD_DIR/graal-enterprise/graal-enterprise",
         VM_SUITE_HOME: "$BUILD_DIR/graal-enterprise/vm-enterprise",
       },
     },
@@ -441,9 +428,9 @@ local part_definitions = {
     compiler_metrics: {
       # Run all metrics benchmarks: hello and compile-mandelbrot
       benchmarks+:: [
-        ["allocation", "--", "--graal"],
-        ["minheap", "--", "--graal"],
-        ["time", "--", "--graal"],
+        "allocation",
+        "minheap",
+        "time",
       ]
     },
 

doc/contributor/benchmarking.md

@@ -2,6 +2,14 @@
 
 ## Benchmarking with the GraalVM Compiler
 
+First build TruffleRuby and include the GraalVM Compiler:
+
+```bash
+$ jt build --graal
+```
+
+Then run the benchmark, for instance:
+
 ```bash
 $ jt benchmark bench/classic/mandelbrot.rb --simple
 ```
@@ -11,10 +19,16 @@ for the first few iterations).
 
 # Benchmarking without the GraalVM Compiler
 
-You can turn off the GraalVM Compiler if you want using `--no-graal`.
+You can turn off the GraalVM Compiler if you want, by not including it in the GraalVM build:
 
 ```bash
-$ jt benchmark --no-graal bench/classic/mandelbrot.rb --simple
+$ jt build
+```
+
+It's the same command to run the benchmark, for instance:
+
+```bash
+$ jt benchmark bench/classic/mandelbrot.rb --simple
 ```
 
 You can benchmark an entirely different implementation using the

doc/contributor/building-graal.md

@@ -6,32 +6,16 @@ when developing TruffleRuby, or possibly to modify them, or to use developer
 tools that come with Graal such as the Ideal Graph Visualizer, then you should
 install Graal locally.
 
-## Installing Graal with jt
+## Building Graal with jt
 
-Our workflow tool can install Graal automatically under the directory
-`../graal/compiler` with:
+Our workflow tool can build Graal automatically with:
 
 ```bash
-$ jt install graal
+$ jt build --graal
 ```
 
-You can then use `jt` to run TruffleRuby with Graal.
+You can then use `jt` as normal to run TruffleRuby with Graal.
 
 ```bash
-$ jt ruby --graal ...
-```
-
-## Building Graal manually
-
-If you want to build the GraalVM Compiler by yourself, follow the instructions
-in the Graal repository.
-
-https://github.com/oracle/graal
-
-You can then set the `GRAAL_HOME` environment variable to the location of the
-`compiler` directory inside the `graal` repository and use `jt` to run
-TruffleRuby.
-
-```bash
-$ GRAAL_HOME=.../graal/compiler jt ruby --graal ...
+$ jt ruby ...
 ```

doc/contributor/workflow.md

@@ -140,8 +140,8 @@ Ruby command line options and arguments can also be set in `RUBYOPT`.
 
 ## Running with Graal
 
-See the document on [build graal](building-graal.md), and then run `jt ruby`
-with the `--graal` option.
+See the document on [building graal](building-graal.md), and then run `jt ruby`
+as normal.
 
 We have flags in `jt` to set some options, such as `--trace` for
 `--vm.Dgraal.TraceTruffleCompilation=true` and `--igv` for

doc/samples/can-we-fold-yet.rb

@@ -10,10 +10,7 @@
 
 require 'readline'
 
-unless TruffleRuby.jit?
-  puts 'You need the GraalVM Compiler to run this'
-  exit
-end
+abort 'You need the GraalVM Compiler to run this' unless TruffleRuby.jit?
 
 puts 'Can Truffle constant fold yet?'
 

mx.truffleruby/mx_truffleruby_benchmark.py

@@ -269,8 +269,6 @@ def filterLines(self, lines):
 
     def runBenchmark(self, benchmark, bmSuiteArgs):
         arguments = ['benchmark']
-        if 'MX_NO_GRAAL' in os.environ:
-            arguments.extend(['--no-graal'])
         arguments.extend(['--simple', '--elapsed', '--iterations'])
         arguments.extend(['--time', str(self.time())])
         if ':' in benchmark:
@@ -586,8 +584,8 @@ def name(self):
 
     def runBenchmark(self, benchmark, bmSuiteArgs):
         arguments = ['ruby']
-        if 'MX_NO_GRAAL' not in os.environ and not bmSuiteArgs:
-            arguments.extend(['--graal', '--vm.Dgraal.TruffleCompilationExceptionsAreFatal=true'])
+        if not bmSuiteArgs:
+            arguments.extend(['--vm.Dgraal.TruffleCompilationExceptionsAreFatal=true'])
         arguments.extend(['bench/servers/' + benchmark + '.rb'])
 
         server = BackgroundJT(arguments + bmSuiteArgs)

test/truffle/compiler/can-we-fold-yet.sh

@@ -2,7 +2,7 @@
 
 source test/truffle/common.sh.inc
 
-jt ruby --graal --vm.Dgraal.TruffleIterativePartialEscape=true doc/samples/can-we-fold-yet.rb < test/truffle/compiler/can-we-fold-yet/input.txt > actual.txt
+jt ruby --vm.Dgraal.TruffleIterativePartialEscape=true doc/samples/can-we-fold-yet.rb < test/truffle/compiler/can-we-fold-yet/input.txt > actual.txt
 
 if ! cmp test/truffle/compiler/can-we-fold-yet/expected.txt actual.txt
 then

test/truffle/compiler/compile-immediately.sh

@@ -2,4 +2,4 @@
 
 source test/truffle/common.sh.inc
 
-jt ruby --graal --stress --trace -e "puts 'hello'"
+jt ruby --stress --trace -e "abort 'not running the GraalVM Compiler' unless TruffleRuby.jit?; puts 'hello'"

test/truffle/compiler/osr.sh

@@ -2,4 +2,4 @@
 
 source test/truffle/common.sh.inc
 
-jt ruby --graal --vm.Dgraal.TruffleCompilationExceptionsAreFatal=true test/truffle/compiler/osr/osr.rb
+jt ruby --vm.Dgraal.TruffleCompilationExceptionsAreFatal=true test/truffle/compiler/osr/osr.rb