)]}'
{
  "commit": "3e6dc39ed7a82facdc2b78c6f7a73ce5beff3cdf",
  "tree": "aee5cb0050a15b1353b785449426e2e55ae6331a",
  "parents": [
    "3293b4f66797bb2ccd3dbbf170249556fc16ced7"
  ],
  "author": {
    "name": "Hans-Peter Nilsson",
    "email": "hp@axis.com",
    "time": "Mon Feb 14 23:50:29 2022 +0100"
  },
  "committer": {
    "name": "Hans-Peter Nilsson",
    "email": "hp@bitrange.com",
    "time": "Mon Feb 14 23:50:29 2022 +0100"
  },
  "message": "sim/testsuite: Set global_cc_os also when no compiler is found\n\nIf we don\u0027t set this variable, it doesn\u0027t exist, and using \"#progos:\"\nin an assembler-file will cause an error rather than just skipping the\ntest, viz:\n\nRunning /src/sim/testsuite/cris/hw/rv-n-cris/rvc.exp ...\nERROR: tcl error sourcing /src/sim/testsuite/cris/hw/rv-n-cris/rvc.exp.\nERROR: can\u0027t read \"global_cc_os\": no such variable\n    while executing\n\"if { $opts(progos) !\u003d \"\" \u0026\u0026 $opts(progos) !\u003d $global_cc_os } {\n\tuntested $subdir/$name\n\treturn\n    }\"\n    (procedure \"run_sim_test\" line 102)\n\nNeither the commit introducing progos, nor the top comment\nin run_sim_test, mentions progos as intended only for C\ntests, or that its use must be gated on $global_cc_works !\u003d\n0, so (not) setting it in the no-working-compiler path seems\njust overlooked.\n\nAllowing it to be used for assembler tests makes it usable\nfor e.g. an always-false predicate and in expressions in\n.exp files without gating on $global_cc_works !\u003d 0.\n\nWith this patch, global_cc_os is set to \"\", just as for \"unknown OS\".\n\n    sim/testsuite:\n\t* lib/sim-defs.exp (sim_init_toolchain): Set global_cc_os also when\n\tno working target C compiler is found.\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "2cf739b3e32e6699dd95f8b42c63f762cd0ef184",
      "old_mode": 33188,
      "old_path": "sim/testsuite/lib/sim-defs.exp",
      "new_id": "3586fa550765bfae33c04f6ca5062757db9c6a8e",
      "new_mode": 33188,
      "new_path": "sim/testsuite/lib/sim-defs.exp"
    }
  ]
}