[PATCH] D22989: AMDGPU: Set sizes on control flow pseudos

Nicolai Hähnle via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 1 06:36:30 PDT 2016


nhaehnle added a subscriber: nhaehnle.
nhaehnle added a comment.

Out of curiosity, what's the purpose of this? Does it affect heuristics somewhere?

Depending on what the semantics are supposed to be, you may want to change SI_ELSE to 8 instead of 12, since the most likely lowering requires only two instructions. (Of course, the maximum is still 12.)

Apart from the SI_ELSE point (which depends on the intended semantics here), LGTM.


https://reviews.llvm.org/D22989





More information about the llvm-commits mailing list