[Bug plugins/57852] New: lib/plugin-support.exp incorrectly sets PLUGINCC to compilers in prev-gcc breaks testing on lean bootstrap

[Bug plugins/57852] New: lib/plugin-support.exp incorrectly sets PLUGINCC to compilers in prev-gcc breaks testing on lean bootstrap

[howarth at nitro dot med.uc.edu]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57852

            Bug ID: 57852
           Summary: lib/plugin-support.exp incorrectly sets PLUGINCC to
                    compilers in prev-gcc breaks testing on lean bootstrap
           Product: gcc
           Version: unknown
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: plugins
          Assignee: unassigned at gcc dot gnu.org
          Reporter: howarth at nitro dot med.uc.edu

After switching my normal gcc48 packaging build to use lean bootstraps, I
noticed a number of test suite failures like...

FAIL: gcc.dg/plugin/selfassign.c compilation
FAIL: gcc.dg/plugin/ggcplug.c compilation
FAIL: gcc.dg/plugin/one_time_plugin.c compilation
FAIL: gcc.dg/plugin/start_unit_plugin.c compilation
FAIL: gcc.dg/plugin/finish_unit_plugin.c compilation

this is due to PLUGIN_CC being set to the compilers in the prev-gcc
subdirectory rather than those in the gcc subdirectory as seen from one of the
failing tests..

Executing on build:
/sw/src/fink.build/gcc48-4.8.1-1000/darwin_objdir/./prev-gcc/xg++
-B/sw/src/fink.build/gcc48-4.8.1-1000/darwin_objdir/./prev-gcc/
-B/sw/lib/gcc4.8/x86_64-apple-darwin13.0.0/bin/ -nostdinc++
-B/sw/src/fink.build/gcc48-4.8.1-1000/darwin_objdir/prev-x86_64-apple-darwin13.0.0/libstdc++-v3/src/.libs
-B/sw/src/fink.build/gcc48-4.8.1-1000/darwin_objdir/prev-x86_64-apple-darwin13.0.0/libstdc++-v3/libsupc++/.libs
-I/sw/src/fink.build/gcc48-4.8.1-1000/darwin_objdir/prev-x86_64-apple-darwin13.0.0/libstdc++-v3/include/x86_64-apple-darwin13.0.0
-I/sw/src/fink.build/gcc48-4.8.1-1000/darwin_objdir/prev-x86_64-apple-darwin13.0.0/libstdc++-v3/include
-I/sw/src/fink.build/gcc48-4.8.1-1000/gcc-4.8.1/libstdc++-v3/libsupc++
-L/sw/src/fink.build/gcc48-4.8.1-1000/darwin_objdir/prev-x86_64-apple-darwin13.0.0/libstdc++-v3/src/.libs
-L/sw/src/fink.build/gcc48-4.8.1-1000/darwin_objdir/prev-x86_64-apple-darwin13.0.0/libstdc++-v3/libsupc++/.libs
-g -O2
/sw/src/fink.build/gcc48-4.8.1-1000/gcc-4.8.1/gcc/testsuite/gcc.dg/plugin/selfassign.c
-I. -I/sw/src/fink.build/gcc48-4.8.1-1000/gcc-4.8.1/gcc/testsuite
-I/sw/src/fink.build/gcc48-4.8.1-1000/gcc-4.8.1/gcc/testsuite/../../gcc
-I/sw/src/fink.build/gcc48-4.8.1-1000/darwin_objdir/gcc/testsuite/gcc/../../../gcc
 -I/sw/src/fink.build/gcc48-4.8.1-1000/gcc-4.8.1/gcc/testsuite/../../include
-I/sw/src/fink.build/gcc48-4.8.1-1000/gcc-4.8.1/gcc/testsuite/../../libcpp/include
 -I/sw/include -I/sw/include 
-I/sw/src/fink.build/gcc48-4.8.1-1000/darwin_objdir/gcc/testsuite/gcc/../../../intl
-O -DIN_GCC -fPIC -shared -undefined dynamic_lookup -o selfassign.so   
(timeout = 300)
set_ld_library_path_env_vars:
ld_library_path=:/sw/src/fink.build/gcc48-4.8.1-1000/darwin_objdir/gcc:/sw/src/fink.build/gcc48-4.8.1-1000/darwin_objdir/x86_64-apple-darwin13.0.0/i386/libsanitizer/asan/.libs
FAIL: gcc.dg/plugin/selfassign.c compilation

Please set PLUGIN_CC to the final build of the compilers in the gcc
subdirectory.

[Bug plugins/57852] lib/plugin-support.exp incorrectly sets PLUGINCC to compilers in prev-gcc breaks testing on lean bootstrap

[ebotcazou at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57852

Eric Botcazou <ebotcazou at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
                 CC|                            |ebotcazou at gcc dot gnu.org
         Resolution|---                         |WONTFIX

--- Comment #1 from Eric Botcazou <ebotcazou at gcc dot gnu.org> ---
> Please set PLUGIN_CC to the final build of the compilers in the gcc subdirectory.

No, that would be wrong, PLUGINCC should be set to the compiler that was used
to build the compiler, otherwise configuration mismatches can arise, for
example if you don't bootstrap.