On 01/24/2011 11:40 AM, Steve Ellcey wrote: > On Sat, 2011-01-22 at 12:26 -0800, Richard Henderson wrote: >> On 01/22/2011 10:48 AM, Sergei Trofimovich wrote: >>> I've attached dirty patch. It has not very nice comments, tabs and spaces >>> yet. >> >> Steve perhaps should weigh in here... > > I am not very familiar with AUTO_PIC and NO_PIC. It would be nice to > implement the TODO: parts as part of this implementation to make it more > complete. Would it make sense for TARGET_AUTO_PIC to automatically set > ia64_cmodel to CMODEL_LARGE? It looks like everywhere we check for > ia64_cmodel equal to CMODEL_LARGE, we also check for TARGET_AUTO_PIC so > this could simplify the logic.
Yes, that's what I suggested later in the message. r~