[PATCH] D40960: Fix for bug PR35549 - [X86] Repeated schedule comments

[Andrew V. Tischenko via Phabricator via llvm-commits]

avt77 added inline comments.


================
Comment at: lib/Target/X86/X86MCInstLower.cpp:2007
   MCInstLowering.Lower(MI, TmpInst);
+  TmpInst.setFlags(MI->getFlags());
 
----------------
RKSimon wrote:
> avt77 wrote:
> > RKSimon wrote:
> > > Won't this copy other flags as well? Is that safe?
> > Yes, that's exactly what we need here.
> So is this a separate issue that needs tests and possibly a separate preliminary patch? Are you missing prefix flags data because of this?
I was completely wrong here: I mixed Flags from MCInst and MachineInstr. It's redone now. 


https://reviews.llvm.org/D40960