GCN: Tag '-march=[...]', '-mtune=[...]' as 'Negative' of themselves [PR112669] (was: [gcn][patch] Add -mgpu option and plumb in assembler/linker)

2023-11-24 Thread Thomas Schwinge
Hi! On 2017-06-21T11:06:24+0100, Andrew Stubbs wrote: > --- a/gcc/config/gcn/gcn.opt > +++ b/gcc/config/gcn/gcn.opt > +march= > +Target RejectNegative Joined ToLower Enum(gpu_type) Var(gcn_arch) > Init(PROCESSOR_CARRIZO) > +Specify the name of the target GPU. > + > +mtune= > +Target RejectNegat

Re: [gcn][patch] Add -mgpu option and plumb in assembler/linker

2017-06-21 Thread Andrew Stubbs
On 06/06/17 20:52, Andrew Stubbs wrote: Thomas objects to the new option, and after talking with him the reasoning seems sound. GCC has been moving away from -mcpu in any case, so I guess I'll put -march and -mtune back, and use those for the same purpose. I'll commit the patch with those cha

Re: [gcn][patch] Add -mgpu option and plumb in assembler/linker

2017-06-06 Thread Andrew Stubbs
On 29/05/17 18:27, Martin Jambor wrote: I apologize for taking so long to reply, I was traveling for two past weeks and just before that we suffered some local infrastructure issues that prevented me from working on this too. And I've just been on vacation for a week. :-) On Fri, Apr 28, 2017

Re: [gcn][patch] Add -mgpu option and plumb in assembler/linker

2017-06-01 Thread Thomas Schwinge
Hi! Sorry for the late reply. On Fri, 28 Apr 2017 18:06:39 +0100, Andrew Stubbs wrote: > 3. Add -mgpu option and corresponding --with-gpu. I've deliberately used > "gpu" instead of "cpu" because I want offloading compilers to be able to > say "-mcpu=foo -foffload=-mgpu=bar", or even have the h

Re: [gcn][patch] Add -mgpu option and plumb in assembler/linker

2017-05-29 Thread Martin Jambor
Hello Andrew, I apologize for taking so long to reply, I was traveling for two past weeks and just before that we suffered some local infrastructure issues that prevented me from working on this too. On Fri, Apr 28, 2017 at 06:06:39PM +0100, Andrew Stubbs wrote: > This patch, for the "gcn" branch

Re: [gcn][patch] Add -mgpu option and plumb in assembler/linker

2017-05-16 Thread Andrew Stubbs
On 02/05/17 18:08, Martin Jambor wrote: 1. Add specs to drive the LLVM assembler and linker. It requires them to be installed as "as" and "ld", under $target/bin, but then the compiler Just Works with these specs. ...is that I do not have llvm linker at hand and without it I did not manage to m

Re: [gcn][patch] Add -mgpu option and plumb in assembler/linker

2017-05-03 Thread Andrew Stubbs
On 02/05/17 18:08, Martin Jambor wrote: Hi Andrew, sorry for replying only now but yesterday was public holiday here and I am still only in the process of recovering from a long weekend. No problem, the UK had the same. :-) While the only objection I have is the C++ style comment in config/g

Re: [gcn][patch] Add -mgpu option and plumb in assembler/linker

2017-05-02 Thread Martin Jambor
Hi Andrew, sorry for replying only now but yesterday was public holiday here and I am still only in the process of recovering from a long weekend. While the only objection I have is the C++ style comment in config/gcn/gcn.c, another problem, for me at least... On Fri, Apr 28, 2017 at 06:06:39PM

Re: [gcn][patch] Add -mgpu option and plumb in assembler/linker

2017-04-28 Thread Joseph Myers
On Fri, 28 Apr 2017, Andrew Stubbs wrote: > 3. Add -mgpu option and corresponding --with-gpu. I've deliberately used "gpu" > instead of "cpu" because I want offloading compilers to be able to say > "-mcpu=foo -foffload=-mgpu=bar", or even have the host compiler just > understand -mgpu and DTRT. C

[gcn][patch] Add -mgpu option and plumb in assembler/linker

2017-04-28 Thread Andrew Stubbs
This patch, for the "gcn" branch, does three things: 1. Add specs to drive the LLVM assembler and linker. It requires them to be installed as "as" and "ld", under $target/bin, but then the compiler Just Works with these specs. 2. Switch to HSACO format version 2, and have the assembler auto-s