Re: [PATCH] [Microblaze]: PIC Data Text Relative

2018-05-05 Thread Andrew Sadek
ith -fPIE/-fPIC as mentioned before. On Thu, May 3, 2018 at 7:13 PM, Andrew Sadek <andrew.sadek...@gmail.com> wrote: > --resend Michael's reply > > On Mon, Apr 30, 2018 at 1:19 PM, Michael Eager <ea...@eagerm.com> wrote: >> >> >> Applied -- Committed re

Re: [PATCH] [Microblaze]: PIC Data Text Relative

2018-05-03 Thread Andrew Sadek
--resend Michael's reply On Mon, Apr 30, 2018 at 1:19 PM, Michael Eager wrote: > > Applied -- Committed revision 259758. > > Andrew -- Please re-run GCC regression test to verify that nothing > was lost in the editing. > > > > -- > Michael Eagerea...@eagerm.com > 1960 Park

Re: [PATCH] [Microblaze]: PIC Data Text Relative

2018-04-27 Thread Andrew Sadek
Thanks Michael. Please find below updated Change Log. 2018-04-27 Andrew Sadek <andrew.sadek...@gmail.com> Microblaze Target: PIC data text relative * gcc/config/microblaze/microblaze.opt: add new option -mpic-is-data-text-relative. * gcc/config/microblaze/microblaze-pr

Re: [PATCH] [Microblaze]: PIC Data Text Relative

2018-04-19 Thread Andrew Sadek
On Wed, Apr 18, 2018 at 6:57 PM, Michael Eager wrote: > > Hi Andrew -- > > Check indents in the following files: > (Make sure that your tabs are set at 8 chars.) > --- gcc/config/microblaze/microblaze.c > --- gcc/config/microblaze/microblaze.md > I have re-run

Re: [PATCH] [Microblaze]: PIC Data Text Relative

2018-04-18 Thread Andrew Sadek
Hello Michael, I have attached the patch as well, (the same as in : https://github.com/andrewsadek/microblaze-pic-data-text-rel/blob/pic_data_text_rel/PATCH_BUNDLE/gcc.patch) On Thu, Mar 22, 2018 at 8:37 PM, Andrew Sadek <andrew.sadek...@gmail.com> wrote: > Hello Michael, > >

Re: [PATCH] [Microblaze]: PIC Data Text Relative

2018-03-22 Thread Andrew Sadek
: - 2018-03-22 Andrew Sadek <andrew.sadek...@gmail.com> Microblaze Target: PIC data text relative * gcc/config/microblaze/microblaze.opt: add new option -mpic-data-text-rel. * gcc/config/microblaze/microblaze-protos.h (microblaze_constant_address_p): Add microblaze_constant_add

Re: [PATCH] [Microblaze]: PIC Data Text Relative

2018-03-20 Thread Andrew Sadek
uite, so the tests would not fail. It's just nice to have the > > test suite updated when new options are added. > > > > On 03/19/2018 01:07 PM, Michael Eager wrote: > >> Hi Andrew -- > >> > >> Good work. > >> > >> Please submit your

Re: [PATCH] [Microblaze]: PIC Data Text Relative

2018-03-18 Thread Andrew Sadek
aze, expectation is r0 or r13 as a base register which is not the case here 'r20', and these tests also fails with -fPIE. Moreover, the test cases that failed due 'unresolved GLOBAL_OFFSET_TABLE' now pass as we do not have GOT in data text relative. Please tell me if any adjustments needed or If I need to

Re: [PATCH] [Microblaze]: PIC Data Text Relative

2018-03-13 Thread Andrew Sadek
is different, anyway I shall give it a try and send you the results with the new options. On Tue, Mar 13, 2018 at 8:42 AM, Michael Eager <ea...@eagerm.com> wrote: > On 03/12/18 23:10, Andrew Sadek wrote: > >> _Command for running testsuite:_ >> >> /make -k check-gcc R

Re: [PATCH] [Microblaze]: PIC Data Text Relative

2018-03-13 Thread Andrew Sadek
croblazeel-softmmu/xilinx.ld -lm -o ./va-arg-15.exe(timeout = 300)* On Mon, Mar 12, 2018 at 4:30 PM, Michael Eager <ea...@eagerm.com> wrote: > On 03/12/18 06:19, Andrew Sadek wrote: > > >> >> On Mon, Mar 5, 2018 at 9:21 PM, Michael Eager <ea...@eagerm.com > e

Re: [PATCH] [Microblaze]: PIC Data Text Relative

2018-03-12 Thread Andrew Sadek
On Mon, Mar 5, 2018 at 9:21 PM, Michael Eager <ea...@eagerm.com> wrote: > On 03/02/2018 08:12 AM, Andrew Sadek wrote: > >> Hello Michael, >> >> I tried running the whole GCC test suite on the current head (without my >> patch) along with 'microblaze-qem

Re: [PATCH] [Microblaze]: PIC Data Text Relative

2018-03-02 Thread Andrew Sadek
program execution. Is there some command to be passed to QEMU in order make system shut down after program termination with its exit code ? On Tue, Feb 27, 2018 at 10:13 AM, Andrew Sadek <andrew.sadek...@gmail.com> wrote: > Thanks Micheal for your response. > I shall re-submit patche

Re: [PATCH] [Microblaze]: PIC Data Text Relative

2018-02-27 Thread Andrew Sadek
Thanks Micheal for your response. I shall re-submit patches separately after re-running the whole GCC Test suite and re-checking code conventions. For sending to gdb-patches, it was a conflict from my side as actually I thought it is also for binutils. On Tue, Feb 27, 2018 at 2:07 AM, Michael

Re: [PATCH] [Microblaze]: PIC Data Text Relative

2018-02-26 Thread Andrew Sadek
Change logs below. There is a problem in attaching the bundle. I get permanent error from sourceware.org GCC ChangeLog: Microblaze Target: PIC data text relative * gcc/config/microblaze/microblaze.opt: add new option -mpic-data-text-rel. * gcc/config/microblaze/microblaze-protos.h