This is a refactoring that will make future improvements to this pass easier.
This change should not change the behavior of the pass.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
- Build Status
Buildable 35866 Build 35865: arc lint + arc unit
Event Timeline
llvm/lib/Target/AMDGPU/SILoadStoreOptimizer.cpp | ||
---|---|---|
533 | The reason I didn't do it this way originally was because I would also need to pass in TTI and STI to the function or to the constructor for CombineInfo, but I don't mind turning it into a member function. |
Move the default branch to the end for readability? (I realize that's what the original code looks like...)