Uploaded image for project: 'JDK'
  1. JDK
  2. JDK-8252237

C2: Call to compute_separating_interferences has wrong argument order

XMLWordPrintable

    • b21

      CLion static analyzer highlights this oddity.

      PhaseConservativeCoalesce::copy_copy calls compute_separating_interferences with [breaking the lines to make the grouping clear]:

      reg_degree = compute_separating_interferences(
      dst_copy, src_copy,
      b, bindex,
      rm,
      rm_size, reg_degree, // <--- look here
      lr1, lr2
      );

      ...yet the definition is [again, breaking the lines]:

      uint PhaseConservativeCoalesce::compute_separating_interferences(
      Node *dst_copy, Node *src_copy,
      Block *b, uint bindex,
      RegMask &rm,
      uint reg_degree, uint rm_size, // <--- different order
      uint lr1, uint lr2
      ) {

            rraghavan Rahul Raghavan
            shade Aleksey Shipilev
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

              Created:
              Updated:
              Resolved: