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 =