Some more background: This list is used to check the clobber list, and since ms inline asm doesn't have explicit clobber lists, AsmParser::parseMSInlineAsm() adds all touched registers to the clobber list.
This change would also permit the segment registers in the clobber lists for gcc registers. A different fix would be to change parseMSInlineAsm() to not add segment registers as clobbers. rnk tells me he likes that approach better, so I'll do that instead. On Thu, Jul 17, 2014 at 12:06 PM, Eric Christopher <[email protected]> wrote: > Seems reasonable. > > -eric > > On Thu, Jul 17, 2014 at 11:55 AM, Nico Weber <[email protected]> wrote: > > Hi, > > > > the attached patch fixes PR20343 by adding the segment registers to the > x86 > > register list in Targets.cpp. > > > > Nico > > > > _______________________________________________ > > cfe-commits mailing list > > [email protected] > > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits > > >
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
