[LLVMbugs] [Bug 23933] New: Convert inline asm to MCInsts (nuke the pre-MC AsmPrinter)

[bugzilla-daemon at llvm.org]

https://llvm.org/bugs/show_bug.cgi?id=23933

            Bug ID: 23933
           Summary: Convert inline asm to MCInsts (nuke the pre-MC
                    AsmPrinter)
           Product: libraries
           Version: trunk
          Hardware: PC
                OS: All
            Status: NEW
          Severity: normal
          Priority: P
         Component: MC
          Assignee: unassignedbugs at nondot.org
          Reporter: dexonsmith at apple.com
                CC: llvmbugs at cs.uiuc.edu, pete.cooper at gmail.com,
                    rafael.espindola at gmail.com
    Classification: Unclassified

Quoting Rafael in
http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20150622/283583.html
--
The way inline asm is implemented is

* We kept most of the old (pre-MC) assembly printer around.
* We use it to print/expand inline asm to a string.
* We parse that string.

So anything that can show up in inline asm needs a name by which we
can find it again.

I think the way to fix this is to directly transition from the inline
asm pattern "bsr $0,%eax" to a MCInst with an operand pointing to
whatever we created for $0. That will let us nuke the old printer and
save memory by not creating names just in case they are used in inline
asm.
--

Filing this so we don't forget the plan.  Once that's done, we should bring
r240130/r240302 back, dropping names from temp symbols.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-bugs/attachments/20150624/c63351fd/attachment.html>