Ticket #11109 (closed: fixed)

Opened 6 years ago

Last modified 5 years ago

PoldiSpectrumConstantBackground causes a compilation error with gcc 4.9.1

Reported by: Michael Wedel Owned by: Michael Wedel
Priority: major Milestone: Release 3.4
Component: Diffraction Keywords: POLDI
Cc: saviciat@…, federico.montesino-pouzols@… Blocked By:
Blocking: Tester: Andrei Savici

Description

When compiling Mantid with gcc 4.9.1 (for example in Ubuntu 14.10), the compiler gives the following error:

Framework/SINQ/test/PoldiSpectrumConstantBackgroundTest.cpp: In member function ‘void PoldiSpectrumConstantBackgroundTest::testParameterCount()’:
   Framework/SINQ/test/PoldiSpectrumConstantBackgroundTest.cpp:40:80: internal compiler error: in possible_polymorphic_call_targets, at ipa-devirt.c:1557
    } testDescription_suite_PoldiSpectrumConstantBackgroundTest_testPoldiFunction1D;

Experiments revealed that making the inheritance virtual gets rid of the error.

Change History

comment:1 Changed 6 years ago by Michael Wedel

  • Status changed from new to verify
  • Resolution set to fixed

This is being verified as pull request #231.

comment:2 Changed 6 years ago by Andrei Savici

  • Status changed from verify to verifying
  • Tester set to Andrei Savici

comment:3 Changed 6 years ago by Michael Wedel

Testing information This needs to be verified by someone with gcc 4.9.1, because in the gcc version used in Ubuntu 14.04 this did not cause the error message. So please make sure everything compiles with gcc 4.9.1 and the PSISINQTEST unit tests still pass, as well as the system tests.

comment:4 Changed 6 years ago by Michael Wedel

Jenkins retest this please

comment:5 Changed 6 years ago by Andrei Savici

  • Status changed from verifying to closed

Merge pull request #231 from mantidproject/11109_fix_gcc_491_error_poldi_constant_backgroun

Fix error caused by PoldiSpectrumConstantBackground with some compilers

Full changeset: 4742c0682dc1ff5941afedcafdd8e99a9c0b3029

comment:6 Changed 6 years ago by Andrei Savici

Works on Ubuntu 14.10

comment:7 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 11948

Note: See TracTickets for help on using tickets.