[PATCH] D49671: [SchedModel] Propagate read advance cycles to implicit operands outside instruction descriptor

Brendon Cahoon via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 26 17:58:45 PDT 2018


bcahoon added a comment.

In https://reviews.llvm.org/D49671#1277445, @kparzysz wrote:

> Hexagon packets (bundles) have 4 slots, numbered 0..3. Each one of the three instructions (2 x `S2_extractu`, and `PS_call_nr`) can only go in slots 2 or 3, so something went horribly wrong.
>
> As it is now, this is very bad, so please do not commit this until we figure this out.


I committed a patch that fixes this problem, so ps_call_nr.ll should pass now.


https://reviews.llvm.org/D49671





More information about the llvm-commits mailing list