Add and improve debuginfo tests for Windows

[jdupak-ms]

Adds new test for closures and function pointers.
Improves robustness of existing tests by sorting wildcard matched outputs.

[rustbot]

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @Mark-Simulacrum (or someone else) some time within the next two weeks.

Please see the contribution instructions for more information. Namely, in order to ensure the minimum review times lag, PR authors and assigned reviewers should ensure that the review label (S-waiting-on-review and S-waiting-on-author) stays updated, invoking these commands when appropriate:

• @rustbot author: the review is finished, PR author should check the comments and take action accordingly
• @rustbot review: the author is ready for a review, this PR will be queued again in the reviewer's queue

[jdupak-ms]

CC @wesleywiser

[jieyouxu]

Do these tests need to be gated by some kind of minimum cdb version?

[jdupak-ms]

Do these tests need to be gated by some kind of minimum cdb version?

They should not. I will try to get some old cdb and verify.

[jieyouxu]

FTR, that is mostly relevant in periods of times where our CI has like a weird mix of 2 cdb versions, e.g. when github gradually rolls out Windows server version bumps. Probably not needed in this case.

[wesleywiser]

Thanks @jdupak-ms! I don't see anything here that should require a particularly recent version of cdb to work but if you can test on an older version, that would be great!

If not, let us know and we can merge.

[jieyouxu]

I don't really expect a cdb version problem since we're currently not in a cdb transition period, so let's just send this off to see if this passes full CI.

@bors r=@wesleywiser rollup=iffy (debuginfo windows tests)

[bors]

📌 Commit f86ca11 has been approved by wesleywiser

It is now in the queue for this repository.

[jdupak-ms]

I don't really expect a cdb version problem since we're currently not in a cdb transition period, so let's just send this off to see if this passes full CI.

@bors r=@wesleywiser rollup=iffy (debuginfo windows tests)

OK, sorry for the delay. I had trouble with getting the older cdb to be picked up on my machine.

[jieyouxu]

OK, sorry for the delay. I had trouble with getting the older cdb to be picked up on my machine.

Thanks for trying anyway!

[bors]

⌛ Testing commit f86ca11 with merge 7d9573b...

[rust-log-analyzer]

The job aarch64-apple failed! Check out the build log: (web) (plain)

Click to see the possible cause of the failure (guessed by this bot)

fmt check
fmt: skipped untracked file AWSCLIV2.pkg
fmt: checked 5760 files
tidy check
tidy error: /Users/runner/work/rust/rust/tests/debuginfo/type-names.rs:172: line longer than 100 chars
tidy error: /Users/runner/work/rust/rust/tests/debuginfo/lexical-scope-in-if-let.rs:50: line longer than 100 chars
some tidy checks failed
Command has failed. Rerun with -v to see more details.
  local time: Wed Jan  8 15:28:27 UTC 2025
  network time: Wed, 08 Jan 2025 15:28:27 GMT
##[error]Process completed with exit code 1.
Post job cleanup.

[bors]

💔 Test failed - checks-actions