In IndVarSimplify after simplifying and extending loop IVs we call replaceCongruentIVs() function.
This function optionally takes a TTI argument in order to be able to replace narrow IVs uses with truncates of the widest one.
For some reason the TTI wasn't passed to the function in IndVars pass, so it couldn't perform such
transform.
This patch fixes it.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
The majority of the regenerated checks are check's variables renames (these only happen with the patch applied).
The functional impact is only reflected in pr27133.ll and widen-loop-comp.ll files.
Comment Actions
Wow, that's impressive number of noop updates. :) Change in pr27133.ll is OK and widen-loop-comp.ll has improvements so LGTM. Passing there TTI totally makes sense.