search for: r183966

Filing a bug would be a good start, go ahead and cc me and jfb at google.com. Thanks! -eric On Tue, Aug 20, 2013 at 6:10 PM, Gordon Keiser <gkeiser at arxan.com> wrote: > For ARM targets on linux, revision 183966 made Fast ISel default. > Unfortunately, Fast ISel is broken in terms of applying the ARMII::MO_PLT > flags to calls in PIC mode (at least when emitting assembly); it

...pp EconomyT > > > -----Original Message----- > > From: Eric Christopher [mailto:echristo at gmail.com] > > Sent: Tuesday, August 20, 2013 9:47 PM > > To: Gordon Keiser > > Cc: llvmdev at cs.uiuc.edu; JF Bastien > > Subject: Re: [LLVMdev] Broken PLT on ARM from R183966 > > > > Filing a bug would be a good start, go ahead and cc me and > jfb at google.com. > > > > Thanks! > > > > -eric > > > > On Tue, Aug 20, 2013 at 6:10 PM, Gordon Keiser <gkeiser at arxan.com> > > wrote: > > > For ARM target...

...www.arxan.com  Protecting the App EconomyT  > -----Original Message----- > From: Eric Christopher [mailto:echristo at gmail.com] > Sent: Tuesday, August 20, 2013 9:47 PM > To: Gordon Keiser > Cc: llvmdev at cs.uiuc.edu; JF Bastien > Subject: Re: [LLVMdev] Broken PLT on ARM from R183966 > > Filing a bug would be a good start, go ahead and cc me and jfb at google.com. > > Thanks! > > -eric > > On Tue, Aug 20, 2013 at 6:10 PM, Gordon Keiser <gkeiser at arxan.com> > wrote: > > For ARM targets on linux, revision 183966 made Fast ISel default....

For ARM targets on linux, revision 183966 made Fast ISel default. Unfortunately, Fast ISel is broken in terms of applying the ARMII::MO_PLT flags to calls in PIC mode (at least when emitting assembly); it never does this. The normal ISel pass handles this situation correctly so a temporary local change to disable FastISel for linux / NaCl targets is working for me right now. I'm not very

That change seems to fix things here. Thanks! -Gordon From: llvmdev-bounces at cs.uiuc.edu [mailto:llvmdev-bounces at cs.uiuc.edu] On Behalf Of JF Bastien Sent: Wednesday, August 21, 2013 12:53 PM To: Logan Chien Cc: llvmdev at cs.uiuc.edu Subject: Re: [LLVMdev] Broken PLT on ARM from R183966 I'm not very familiar with relocations but your fix looks the same as ARMTargetLowering::LowerCall, so from that perspective it lgtm (but I may be missing something). On Wed, Aug 21, 2013 at 8:04 AM, Logan Chien <tzuhsiang.chien at gmail.com<mailto:tzuhsiang.chien at gmail.com>> w...

...: Eric Christopher [mailto:echristo at gmail.com] >>> >> > Sent: Tuesday, August 20, 2013 9:47 PM >>> >> > To: Gordon Keiser >>> >> > Cc: llvmdev at cs.uiuc.edu; JF Bastien >>> >> > Subject: Re: [LLVMdev] Broken PLT on ARM from R183966 >>> >> > >>> >> > Filing a bug would be a good start, go ahead and cc me and >>> >> > jfb at google.com. >>> >> > >>> >> > Thanks! >>> >> > >>> >> > -eric >>> >&...

...age----- > >> > From: Eric Christopher [mailto:echristo at gmail.com] > >> > Sent: Tuesday, August 20, 2013 9:47 PM > >> > To: Gordon Keiser > >> > Cc: llvmdev at cs.uiuc.edu; JF Bastien > >> > Subject: Re: [LLVMdev] Broken PLT on ARM from R183966 > >> > > >> > Filing a bug would be a good start, go ahead and cc me and > >> > jfb at google.com. > >> > > >> > Thanks! > >> > > >> > -eric > >> > > >> > On Tue, Aug 20, 2013 at 6:10 PM, Gor...

...; > -----Original Message----- >> > From: Eric Christopher [mailto:echristo at gmail.com] >> > Sent: Tuesday, August 20, 2013 9:47 PM >> > To: Gordon Keiser >> > Cc: llvmdev at cs.uiuc.edu; JF Bastien >> > Subject: Re: [LLVMdev] Broken PLT on ARM from R183966 >> > >> > Filing a bug would be a good start, go ahead and cc me and >> > jfb at google.com. >> > >> > Thanks! >> > >> > -eric >> > >> > On Tue, Aug 20, 2013 at 6:10 PM, Gordon Keiser <gkeiser at arxan.com> >&...

...t;> > From: Eric Christopher [mailto:echristo at gmail.com] >> >> > Sent: Tuesday, August 20, 2013 9:47 PM >> >> > To: Gordon Keiser >> >> > Cc: llvmdev at cs.uiuc.edu; JF Bastien >> >> > Subject: Re: [LLVMdev] Broken PLT on ARM from R183966 >> >> > >> >> > Filing a bug would be a good start, go ahead and cc me and >> >> > jfb at google.com. >> >> > >> >> > Thanks! >> >> > >> >> > -eric >> >> > >> >> >...