[LLVMbugs] [Bug 8420] New: MachineInstr constructor does not correctly duplicate AsmPrinterFlags

[bugzilla-daemon at llvm.org]

http://llvm.org/bugs/show_bug.cgi?id=8420

           Summary: MachineInstr constructor does not correctly duplicate
                    AsmPrinterFlags
           Product: libraries
           Version: trunk
          Platform: PC
        OS/Version: Windows NT
            Status: NEW
          Severity: normal
          Priority: P
         Component: Common Code Generator Code
        AssignedTo: unassignedbugs at nondot.org
        ReportedBy: micah.villmow at amd.com
                CC: llvmbugs at cs.uiuc.edu


In MachineInstr.cpp with the MachineInstr ctor
MachineInstr::MachineInstr(MachineFunction &MF, const MachineInstr &MI)

The constructor incorrectly sets AsmPrinterFlags to zero instead of copying the
value that is in the source MachineInstr.

The correct implementation should be:
MachineInstr::MachineInstr(MachineFunction &MF, const MachineInstr &MI)
  : TID(&MI.getDesc()), NumImplicitOps(0),
AsmPrinterFlags(MI.getAsmPrinterFlags()),

-- 
Configure bugmail: http://llvm.org/bugs/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.