ldc icon indicating copy to clipboard operation
ldc copied to clipboard

Move to use new sanitizer passes

Open thewilsonator opened this issue 2 years ago • 2 comments

see #3961

I chose to enable them from LLVM13+ partly because that's when the old ones were deprecated and partly to ensure that the CI tests them. That can be easily changed if desired.

Still to go is replacement for createPGOIndirectCallPromotionLegacyPass.

thewilsonator avatar May 06 '22 02:05 thewilsonator

The sanitizers are probably just a minor aspect of switching to the 'new' pass manager (https://llvm.org/docs/NewPassManager.html). We also need to port our passes (https://forum.dlang.org/thread/[email protected], https://llvm.org/docs/WritingAnLLVMNewPMPass.html) before we can use it.

kinke avatar May 06 '22 08:05 kinke

I think it is better to make new/old passmanager selectable through a runtime option (commandflag) in the transition period. Easier to investigate bugs that way.

JohanEngelen avatar May 06 '22 13:05 JohanEngelen

superseded by https://github.com/ldc-developers/ldc/pull/4105

thewilsonator avatar Sep 15 '22 08:09 thewilsonator