<html><head><meta http-equiv="Content-Type" content="text/html charset=windows-1252"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">Address spaces are also used for the x86 segment registers (fs, gs), but like Tom said for R600, i’m not sure if x86 actually needs any MC level features to implement this.<div><br></div><div>Thanks,</div><div>Pete<br><div><div>On Jun 27, 2013, at 3:10 PM, Tom Stellard <<a href="mailto:tom@stellard.net">tom@stellard.net</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div style="letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;">On Thu, Jun 27, 2013 at 09:43:43AM -0400, Rafael Espíndola wrote:<br><blockquote type="cite">Reviewing<span class="Apple-converted-space"> </span><a href="http://llvm-reviews.chandlerc.com/D1043">http://llvm-reviews.chandlerc.com/D1043</a><span class="Apple-converted-space"> </span>I noticed that<br>implementation of address spaces in MC was a bit messy. Parts of some<br>streamers support it, others parts ignore it and yet others assert<br>that it is 0.<br><br>I wanted to refactor it, but there is a problem: We don't have *any*<br>tests. Is this used only by out of tree targets? What about R600,<br>doesn't it use address spaces? Only for dynamic memory?<br><br></blockquote><br>R600 uses address spaces, but I don't think we currently use any MC<br>features that need to be aware of them. We only emit code and not much<br>else.<br><br>-Tom<br><br><blockquote type="cite">Another option is that all data in object files is in the default<br>address space for all targets and this patch removes dead code.<br>Without tests it is hard to tell.<br><br>If you have an out of tree target that uses address spaces, please try<br>this patch and let me know if it breaks something for you.<br><br>This was added back in r63377 by Sanjiv Gupta from microchip, so it is<br>possible that this was only used by PIC and is really dead code.<br><br>Cheers,<br>Rafael<br></blockquote><br><br><blockquote type="cite">_______________________________________________<br>llvm-commits mailing list<br><a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br><a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br></blockquote><br><br>_______________________________________________<br>llvm-commits mailing list<br><a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br><a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a></div></blockquote></div><br></div></body></html>