Uploaded image for project: 'Harmony'
  1. Harmony
  2. HARMONY-6024

[drlvm][jit] Incorrect generated instruction in peephole opt for MOV

Add voteWatch issue
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Open
    • Major
    • Resolution: Unresolved
    • None
    • None
    • DRLVM
    • None
    • IA32 Windows

    Description

      In the case 1 of peephole opt for MOV in Ia32PeepHole.cpp, the optimization is like following:

      MOV opnd, opnd2
      MOV opnd3, opnd
      =>
      MOV opnd3, opnd2

      The bug is the new MOV instruction is always generated by "newInst" even the original MOV instructions are both pseudo copy, which will cause an assertion error later.

      Attachments

        1. H6024-V1.patch
          3 kB
          Xiaoming Gu

        Issue Links

          is part of

          Bug - A problem which impairs or prevents the functions of the product. HARMONY-6027 [drlvm][jit] An incorrect register assignment in graph coloring register allocation

          • Major - Major loss of function.
          • Open

          Activity

            People

              Unassigned Unassigned
              xiaoming Xiaoming Gu
              Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

                Created:
                Updated:

                Slack

                  Issue deployment