Extract code dealing with branch weights in separate functions.
Details
Details
Diff Detail
Diff Detail
- Repository
- rL LLVM
Event Timeline
Comment Actions
Hi Florian, I will land this change with comment update because I do not understand what exactly comment needs to be fixed.
Feel free to update this review with details what wrong in comment you see and I will land it as follow-up patch.
lib/Transforms/Utils/LoopUnrollPeel.cpp | ||
---|---|---|
390 ↗ | (On Diff #206996) | Yep, it was = a small thing. I just found it a bit inconsistent that ExitWeight and CurHeaderWeight are described differently. I think something like \param[out] ExitWeight The number of times the edge from Latch to Exit block is executed (or taken) would be more consistent with the description for CurHeaderWeight. |