Re: [MPX] Fix for PR77267

2016-09-05 Thread Ilya Enkovich
2016-09-05 12:56 GMT+03:00 Alexander Ivchenko : > Ok, thanks. The full updated patch is below. I also removed the > '--whole-archive' thing from -static-libmpxwrappers case. This version is OK for trunk with proper ChangeLog and after proper testing. > Would that be possible to backport that patc

Re: [MPX] Fix for PR77267

2016-09-05 Thread Alexander Ivchenko
Ok, thanks. The full updated patch is below. I also removed the '--whole-archive' thing from -static-libmpxwrappers case. Would that be possible to backport that patch to 6 branch as well? And could you please also address Sergey's comment on adding weak symbol that he's made in the bugzilla? dif

Re: [MPX] Fix for PR77267

2016-08-31 Thread Ilya Enkovich
2016-08-31 16:37 GMT+03:00 Alexander Ivchenko : > 2016-08-31 12:18 GMT+03:00 Ilya Enkovich : >> 2016-08-30 21:53 GMT+03:00 Alexander Ivchenko : >>> Would something like that count? >>> >>> I did not do the warning thing, cause the problem only appears when >>> you provide the -Wl,-as-needed option

Re: [MPX] Fix for PR77267

2016-08-31 Thread Alexander Ivchenko
2016-08-31 12:18 GMT+03:00 Ilya Enkovich : > 2016-08-30 21:53 GMT+03:00 Alexander Ivchenko : >> Would something like that count? >> >> I did not do the warning thing, cause the problem only appears when >> you provide the -Wl,-as-needed option to the linker. >> (In all other cases warning would be

Re: [MPX] Fix for PR77267

2016-08-31 Thread Ilya Enkovich
2016-08-30 21:53 GMT+03:00 Alexander Ivchenko : > Would something like that count? > > I did not do the warning thing, cause the problem only appears when > you provide the -Wl,-as-needed option to the linker. > (In all other cases warning would be redundant). Are we able to check > that on runtime

Re: [MPX] Fix for PR77267

2016-08-30 Thread H.J. Lu
On Tue, Aug 30, 2016 at 11:53 AM, Alexander Ivchenko wrote: > Would something like that count? > > I did not do the warning thing, cause the problem only appears when > you provide the -Wl,-as-needed option to the linker. > (In all other cases warning would be redundant). Are we able to check > th

Re: [MPX] Fix for PR77267

2016-08-30 Thread Alexander Ivchenko
Would something like that count? I did not do the warning thing, cause the problem only appears when you provide the -Wl,-as-needed option to the linker. (In all other cases warning would be redundant). Are we able to check that on runtime? diff --git a/gcc/config.in b/gcc/config.in index fc3321

Re: [MPX] Fix for PR77267

2016-08-29 Thread Ilya Enkovich
2016-08-25 12:27 GMT+03:00 Alexander Ivchenko : > The attached patched fixes the usage of MPX in presence of > "-Wl,-as-needed" option. 'make checked' on MPX-enabled machine. > > "--push-state" and "--pop-state" are not supported by gold at the > moment. But that's OK because using MPX with gold on

[MPX] Fix for PR77267

2016-08-25 Thread Alexander Ivchenko
The attached patched fixes the usage of MPX in presence of "-Wl,-as-needed" option. 'make checked' on MPX-enabled machine. "--push-state" and "--pop-state" are not supported by gold at the moment. But that's OK because using MPX with gold only recommended in static build. Would that be OK for tru