| Message ID | 12bf7d5c4d3be69800c276969361e256fc3af1f4.1638820430.git.wschmidt@linux.ibm.com |
|---|---|
| State | New |
| Headers | show |
| Series | Remove "old" built-in function infrastructure | expand |
Ping. Thanks! Bill On 12/6/21 2:49 PM, Bill Schmidt via Gcc-patches wrote: > Hi! > > This patch just renames a file and updates the build machinery accordingly. > > Bootstrapped and tested on powerpc64le-linux-gnu with no regressions. Is this > okay for trunk? > > Thanks! > Bill > > 2021-12-02 Bill Schmidt <wschmidt@linux.ibm.com> > > gcc/ > * config/rs6000/rs6000-builtin-new.def: Rename to... > * config/rs6000/rs6000-builtins.def: ...this. > * config/rs6000/rs6000-gen-builtins.c: Adjust header commentary. > * config/rs6000/t-rs6000 (EXTRA_GTYPE_DEPS): Rename > rs6000-builtin-new.def to rs6000-builtins.def. > (rs6000-builtins.c): Likewise. > --- > .../rs6000/{rs6000-builtin-new.def => rs6000-builtins.def} | 0 > gcc/config/rs6000/rs6000-gen-builtins.c | 4 ++-- > gcc/config/rs6000/t-rs6000 | 6 +++--- > 3 files changed, 5 insertions(+), 5 deletions(-) > rename gcc/config/rs6000/{rs6000-builtin-new.def => rs6000-builtins.def} (100%) > > diff --git a/gcc/config/rs6000/rs6000-builtin-new.def b/gcc/config/rs6000/rs6000-builtins.def > similarity index 100% > rename from gcc/config/rs6000/rs6000-builtin-new.def > rename to gcc/config/rs6000/rs6000-builtins.def > diff --git a/gcc/config/rs6000/rs6000-gen-builtins.c b/gcc/config/rs6000/rs6000-gen-builtins.c > index 78b2486aafc..9c61b7d9fe6 100644 > --- a/gcc/config/rs6000/rs6000-gen-builtins.c > +++ b/gcc/config/rs6000/rs6000-gen-builtins.c > @@ -22,7 +22,7 @@ along with GCC; see the file COPYING3. If not see > recognition code for Power targets, based on text files that > describe the built-in functions and vector overloads: > > - rs6000-builtin-new.def Table of built-in functions > + rs6000-builtins.def Table of built-in functions > rs6000-overload.def Table of overload functions > > Both files group similar functions together in "stanzas," as > @@ -125,7 +125,7 @@ along with GCC; see the file COPYING3. If not see > > The second line contains the <bif-id> that this particular instance of > the overloaded function maps to. It must match a token that appears in > - rs6000-builtin-new.def. Optionally, a second token may appear. If only > + rs6000-builtins.def. Optionally, a second token may appear. If only > one token is on the line, it is also used to build the unique identifier > for the overloaded function. If a second token is present, the second > token is used instead for this purpose. This is necessary in cases > diff --git a/gcc/config/rs6000/t-rs6000 b/gcc/config/rs6000/t-rs6000 > index d48a4b1be6c..3d3143a171d 100644 > --- a/gcc/config/rs6000/t-rs6000 > +++ b/gcc/config/rs6000/t-rs6000 > @@ -22,7 +22,7 @@ TM_H += $(srcdir)/config/rs6000/rs6000-builtin.def > TM_H += $(srcdir)/config/rs6000/rs6000-cpus.def > TM_H += $(srcdir)/config/rs6000/rs6000-modes.h > PASSES_EXTRA += $(srcdir)/config/rs6000/rs6000-passes.def > -EXTRA_GTYPE_DEPS += $(srcdir)/config/rs6000/rs6000-builtin-new.def > +EXTRA_GTYPE_DEPS += $(srcdir)/config/rs6000/rs6000-builtins.def > > rs6000-pcrel-opt.o: $(srcdir)/config/rs6000/rs6000-pcrel-opt.c > $(COMPILE) $< > @@ -59,10 +59,10 @@ build/rs6000-gen-builtins$(build_exeext): build/rs6000-gen-builtins.o \ > # For now, the header files depend on rs6000-builtins.c, which avoids > # races because the .c file is closed last in rs6000-gen-builtins.c. > rs6000-builtins.c: build/rs6000-gen-builtins$(build_exeext) \ > - $(srcdir)/config/rs6000/rs6000-builtin-new.def \ > + $(srcdir)/config/rs6000/rs6000-builtins.def \ > $(srcdir)/config/rs6000/rs6000-overload.def > $(RUN_GEN) ./build/rs6000-gen-builtins$(build_exeext) \ > - $(srcdir)/config/rs6000/rs6000-builtin-new.def \ > + $(srcdir)/config/rs6000/rs6000-builtins.def \ > $(srcdir)/config/rs6000/rs6000-overload.def rs6000-builtins.h \ > rs6000-builtins.c rs6000-vecdefines.h >
On Mon, Dec 6, 2021 at 3:49 PM Bill Schmidt <wschmidt@linux.ibm.com> wrote: > > Hi! > > This patch just renames a file and updates the build machinery accordingly. > > Bootstrapped and tested on powerpc64le-linux-gnu with no regressions. Is this > okay for trunk? > > Thanks! > Bill > > 2021-12-02 Bill Schmidt <wschmidt@linux.ibm.com> > > gcc/ > * config/rs6000/rs6000-builtin-new.def: Rename to... > * config/rs6000/rs6000-builtins.def: ...this. > * config/rs6000/rs6000-gen-builtins.c: Adjust header commentary. > * config/rs6000/t-rs6000 (EXTRA_GTYPE_DEPS): Rename > rs6000-builtin-new.def to rs6000-builtins.def. > (rs6000-builtins.c): Likewise. Okay. Thanks, David
diff --git a/gcc/config/rs6000/rs6000-builtin-new.def b/gcc/config/rs6000/rs6000-builtins.def similarity index 100% rename from gcc/config/rs6000/rs6000-builtin-new.def rename to gcc/config/rs6000/rs6000-builtins.def diff --git a/gcc/config/rs6000/rs6000-gen-builtins.c b/gcc/config/rs6000/rs6000-gen-builtins.c index 78b2486aafc..9c61b7d9fe6 100644 --- a/gcc/config/rs6000/rs6000-gen-builtins.c +++ b/gcc/config/rs6000/rs6000-gen-builtins.c @@ -22,7 +22,7 @@ along with GCC; see the file COPYING3. If not see recognition code for Power targets, based on text files that describe the built-in functions and vector overloads: - rs6000-builtin-new.def Table of built-in functions + rs6000-builtins.def Table of built-in functions rs6000-overload.def Table of overload functions Both files group similar functions together in "stanzas," as @@ -125,7 +125,7 @@ along with GCC; see the file COPYING3. If not see The second line contains the <bif-id> that this particular instance of the overloaded function maps to. It must match a token that appears in - rs6000-builtin-new.def. Optionally, a second token may appear. If only + rs6000-builtins.def. Optionally, a second token may appear. If only one token is on the line, it is also used to build the unique identifier for the overloaded function. If a second token is present, the second token is used instead for this purpose. This is necessary in cases diff --git a/gcc/config/rs6000/t-rs6000 b/gcc/config/rs6000/t-rs6000 index d48a4b1be6c..3d3143a171d 100644 --- a/gcc/config/rs6000/t-rs6000 +++ b/gcc/config/rs6000/t-rs6000 @@ -22,7 +22,7 @@ TM_H += $(srcdir)/config/rs6000/rs6000-builtin.def TM_H += $(srcdir)/config/rs6000/rs6000-cpus.def TM_H += $(srcdir)/config/rs6000/rs6000-modes.h PASSES_EXTRA += $(srcdir)/config/rs6000/rs6000-passes.def -EXTRA_GTYPE_DEPS += $(srcdir)/config/rs6000/rs6000-builtin-new.def +EXTRA_GTYPE_DEPS += $(srcdir)/config/rs6000/rs6000-builtins.def rs6000-pcrel-opt.o: $(srcdir)/config/rs6000/rs6000-pcrel-opt.c $(COMPILE) $< @@ -59,10 +59,10 @@ build/rs6000-gen-builtins$(build_exeext): build/rs6000-gen-builtins.o \ # For now, the header files depend on rs6000-builtins.c, which avoids # races because the .c file is closed last in rs6000-gen-builtins.c. rs6000-builtins.c: build/rs6000-gen-builtins$(build_exeext) \ - $(srcdir)/config/rs6000/rs6000-builtin-new.def \ + $(srcdir)/config/rs6000/rs6000-builtins.def \ $(srcdir)/config/rs6000/rs6000-overload.def $(RUN_GEN) ./build/rs6000-gen-builtins$(build_exeext) \ - $(srcdir)/config/rs6000/rs6000-builtin-new.def \ + $(srcdir)/config/rs6000/rs6000-builtins.def \ $(srcdir)/config/rs6000/rs6000-overload.def rs6000-builtins.h \ rs6000-builtins.c rs6000-vecdefines.h
Hi! This patch just renames a file and updates the build machinery accordingly. Bootstrapped and tested on powerpc64le-linux-gnu with no regressions. Is this okay for trunk? Thanks! Bill 2021-12-02 Bill Schmidt <wschmidt@linux.ibm.com> gcc/ * config/rs6000/rs6000-builtin-new.def: Rename to... * config/rs6000/rs6000-builtins.def: ...this. * config/rs6000/rs6000-gen-builtins.c: Adjust header commentary. * config/rs6000/t-rs6000 (EXTRA_GTYPE_DEPS): Rename rs6000-builtin-new.def to rs6000-builtins.def. (rs6000-builtins.c): Likewise. --- .../rs6000/{rs6000-builtin-new.def => rs6000-builtins.def} | 0 gcc/config/rs6000/rs6000-gen-builtins.c | 4 ++-- gcc/config/rs6000/t-rs6000 | 6 +++--- 3 files changed, 5 insertions(+), 5 deletions(-) rename gcc/config/rs6000/{rs6000-builtin-new.def => rs6000-builtins.def} (100%)