Re: [PATCH]: Allow TARGET_SCHED_ADJUST_PRIORITY hook to reduce priority

[John David Anglin]

On 2018-09-17 5:08 AM, Andreas Schwab wrote:
        PR rtl-optimization/85458
        * sel-sched.c (sel_target_adjust_priority): Remove wrong
        assertion.
diff --git a/gcc/sel-sched.c b/gcc/sel-sched.c
index 824f1ec340..1be977d70b 100644
--- a/gcc/sel-sched.c
+++ b/gcc/sel-sched.c
@@ -3330,8 +3330,6 @@ sel_target_adjust_priority (expr_t expr)
    else
      new_priority = priority;
  
-  gcc_assert (new_priority >= 0);
-
    /* If the priority has changed, adjust EXPR_PRIORITY_ADJ accordingly.  */
    EXPR_PRIORITY_ADJ (expr) = new_priority - EXPR_PRIORITY (expr);
  
I added the assert because the hppa implementation of 
TARGET_SCHED_ADJUST_PRIORITY assumes
scheduling priorities are non negative.  If that is not the case, I tend 
to think this should be documented.

It seems ia64 is the only target tripping on the assert.

Dave

--
John David Anglin  dave.ang...@bell.net