@swift-ci create

As a bonus, if you're interested in deleting code, you can try to see if there are other options which aren't being used anywhere.

theindigamer (JIRA User) is this jirid@c03c40f what you're looking for? I did not find the option in FrontendOptions.h, I did find it in IRGenOptions.h and removing it did not break the build for me.

#37397

I went through `FrontendOptions.h` and `IRGenOptions.h` and found 2 options that might be removable.

FrontendOptions.h:
DebugTimeExpressionTypeChecking
jirid@b6776e2

IRGenOptions.h:
DisableLLVMSLPVectorizer
jirid@ed72797

I prepared 2 independent branches with the changes. The default test suite passes after the removal. theindigamer (JIRA User) please let me know whether you consider these removals valid and would like me to open pull requests.

Hmm, that's weird. I think for the first one at least, I thought compiler engineers used it for debugging. Maybe there's a bug somewhere and that variable should actually be used.

For the second one, Clang does support that as an option. So it seems like a bug that it's not being used anywhere. I'll get back to you on Monday/Tuesday on whether these two should be removed or not.

@jirid, the DisableLLVMSLPVectorizer flag can be removed.

The two flags DebugTimeFunctionBodies and DebugTimeExpressionTypeChecking on FrontendOptions should be set when the FrontendOptions are getting parsed (they are not currently set). Then, in ParseTypeCheckerArgs, the type-checker options should copy the value from the FrontendOpts instead of parsing the arguments directly.