)]}'
{
  "commit": "3de2dbe3258b0a8ac64fea3af57b2d2beeedc277",
  "tree": "b4a7146fe0b2b5b91f812bfb484fdd9b89b64d72",
  "parents": [
    "31da5c3604ff991868cc1da7593502993c41af27"
  ],
  "author": {
    "name": "Guinevere Larsen",
    "email": "guinevere@redhat.com",
    "time": "Tue Jan 14 14:28:18 2025 -0300"
  },
  "committer": {
    "name": "Guinevere Larsen",
    "email": "guinevere@redhat.com",
    "time": "Tue Oct 21 16:40:06 2025 -0300"
  },
  "message": "gdb/mdebug: Remove stabs support from mips inferiors\n\nOstensibly, the mdebugread.c is about reading debug information in the\necoff format, but it also supports stabs-in-ecoff according to comments\nin there, and also relied in some stabs facilities for ecoff reading\nitself.  This commit takes the first step in removing stabs support by\nremoving those dependencies from mdebug.  And in order to support\nstabs-in-ecoff, mipsread would also call stabsread_new_init.\n\nRemoving stabs-in-ecoff is trivial, as the code was well demarcated with\ncomments mentioning where stabs was read.  Plus the call to\nstabsread_new_init in mipsread can be trivially removed.\n\nAnother simple removal was the dependence on stabs_end_psymtabs: because\nthe local variables dependencies_used and includes_used were only touched\nby stabs-reading code, they are always 0 in the new version, which means\nwe can find the exact code path that\u0027d be followed in stabs_end_psymtab\nand move the relevant lines to mdebug instead.\n\nAfter all those, the last remaining dependency is when reading a fortran\ncommon block from an inferior compiled by SGI fortran compilers (and\nmaybe more).  The block could have no address, meaning it\u0027d need to be\nfixed after all the minimal symbols have been read.  This was done by\nadding the symbol to the stabs global_sym_chain, then calling\nscan_file_globals to fix them up.  This commit copies all the necessary\ncode for handling the fortran symbols onto mdebug, technically making\nsome code duplication, but since stabsread will be removed soon, this\nshouldn\u0027t be too concerning.\n\nThis change was tested in the compile farm\u0027s mips64 machine (number\n230), where it actually seems to have solved some 50 failures in the\ntestsuite, not including changes in tests from gdb.threads, as those are\noften very racy.  I\u0027m not sure if these were true fixes or racy cases,\nbut since the new version has no newly introduced fails, only fewer of\nthem, I\u0027m inclined to think this change is at least harmless.\n\nAcked-By: Tom Tromey \u003ctom@tromey.com\u003e\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "90c365b85d7b4bd4aee0d13cc557d7b7941e8968",
      "old_mode": 33188,
      "old_path": "gdb/mdebugread.c",
      "new_id": "19d7a3d3e0c62ac9c93c546b912721802ec49c8a",
      "new_mode": 33188,
      "new_path": "gdb/mdebugread.c"
    },
    {
      "type": "modify",
      "old_id": "1d76b8f4eda86f51111c10b75f6ab2f926f29893",
      "old_mode": 33188,
      "old_path": "gdb/mipsread.c",
      "new_id": "943ed9afc5c116e8609be68c5844c64876640152",
      "new_mode": 33188,
      "new_path": "gdb/mipsread.c"
    }
  ]
}