Update to LLVM 17.0.1

[IgorDeepakM]

This is a continuation of #4570.

LLVM version is increased to 17.0.1 instead.

[ponylang-main]

Hi @IgorDeepakM,

The changelog - changed label was added to this pull request; all PRs with a changelog label need to have release notes included as part of the PR. If you haven't added release notes already, please do.

Release notes are added by creating a uniquely named file in the .release-notes directory. We suggest you call the file 4680.md to match the number of this pull request.

The basic format of the release notes (using markdown) should be:

## Title

End user description of changes, why it's important,
problems it solves etc.

If a breaking change, make sure to include 1 or more
examples what code would look like prior to this change
and how to update it to work after this change.

Thanks.

[IgorDeepakM]

I think regression-1118 is close to the threshold of the timeout of 120 seconds. In a test with LLVM 15:
[ OK ] regression-1118 (109404 ms)
I see slightly higher test times with the sanitizer with the LLVM 17 build so this 120 second limit probably passed. Not sure if this depends on LLVM 17 or server load.

[SeanTAllen]

Which LLVM version does this update to 17.0.6?

[IgorDeepakM]

Which LLVM version does this update to 17.0.6?

17.0.1 (tag: llvmorg-17.0.1)

[SeanTAllen]

@IgorDeepakM was there a change that was technically tricky that made it more work to go to the latest llvm 17?

[IgorDeepakM]

@IgorDeepakM was there a change that was technically tricky that made it more work to go to the latest llvm 17?

No, the only changed required for LLVM 17 was removal of LLVMGetGlobalPassRegistry();

  LLVMPassRegistryRef passreg = LLVMGetGlobalPassRegistry();
  LLVMInitializeCore(passreg);
  LLVMInitializeTransformUtils(passreg);
  LLVMInitializeScalarOpts(passreg);
  LLVMInitializeVectorization(passreg);
  LLVMInitializeInstCombine(passreg);
  LLVMInitializeIPO(passreg);
  LLVMInitializeAnalysis(passreg);
  LLVMInitializeIPA(passreg);
  LLVMInitializeCodeGen(passreg);
  LLVMInitializeTarget(passreg);

The rest are LLVM 16 related.

[IgorDeepakM]

There is still an open question if the readnone and readonly removal for functions like pony_ctx, should be replaced with memory(none) and memory(read).

[jemc]

There is still an open question if the readnone and readonly removal for functions like pony_ctx, should be replaced with memory(none) and memory(read).

I have verified that this is correct (in the sync call today).