[PATCH] D133999: [SelectOpti] Restrict load sinking

Sotiris Apostolakis via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 16 12:45:18 PDT 2022


apostolakis added a comment.

In D133999#3796300 <https://reviews.llvm.org/D133999#3796300>, @davidxl wrote:

> Ok -- my confusion was the naming of the SinkPt variable -- it should actually be named 'SI'.   I through the sinkpt is the actual bb after select is expanded into branch.

Oh I see. Naming it as sink-point was indeed confusing. I will change it.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D133999/new/

https://reviews.llvm.org/D133999



More information about the llvm-commits mailing list