Required for D140210 in order to disable FuncSpec at {Os, Oz} optimization levels.
Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
| llvm/test/Transforms/FunctionSpecialization/function-specialization.ll | |||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| 1–2 | For completeness, I'd suggest a few more run lines, so ultimately we cover:
| ||||||||||||||||||||
| llvm/test/Transforms/FunctionSpecialization/function-specialization.ll | ||
|---|---|---|
| 1–2 | Maybe drop -specialize-functions entirely? Is there any benefit to keeping it if we already have the pass parameter? | |
| llvm/test/Transforms/FunctionSpecialization/function-specialization.ll | ||
|---|---|---|
| 1–2 | Agree. | |
| llvm/test/Transforms/FunctionSpecialization/function-specialization.ll | ||
|---|---|---|
| 1–2 | I think this may be a LegacyPM leftover, where pass options didn't exist (or rather, they existed, but not in a form that's directly accessible via opt). | |
| llvm/test/Transforms/FunctionSpecialization/function-specialization.ll | ||
|---|---|---|
| 1–2 | What happens in case like --passes='default<O3>' -specialize-functions (see compiler-crash-58759.ll) ? | |
| llvm/test/Transforms/FunctionSpecialization/function-specialization.ll | ||
|---|---|---|
| 1–2 | Transform to --passes='default<O3>,ipsccp<func-spec>' ? | |
- Removed the cmdline option -specialize-functions.
- Changed the IPSCCPOption 'AllowFuncSpec' from std::optional<bool> to bool.
| llvm/include/llvm/Transforms/IPO/SCCP.h | ||
|---|---|---|
| 38 | Nit: spaces around = | |
Nit: spaces around =