search

laristra / portage

http://portage.lanl.gov
Other
30 stars 8 forks source link

Internal compiler error with Intel 18.0.2 #17

Closed nncarlson closed 6 years ago

nncarlson commented 6 years ago

FYI, I'm seeing this ICE with Intel 18.0.2:

[ 92%] Building CXX object app/swarmapp/CMakeFiles/swarmapp.dir/swarmapp.cc.o
/scratch/portage/portage/portage/support/basis.h(60): internal error: bad pointer
      jet_size={quadratic_sizes[dim], quadratic_sizes[dim]};
                                                           ^

compilation aborted for /scratch/portage/portage/app/swarmapp/swarmapp.cc (code 4)

My cmake line:

cmake -DENABLE_APP_TESTS=True -DCMAKE_INSTALL_PREFIX=`pwd`/../intel18 ..
raovgarimella commented 6 years ago

Neil

This is something from Gary Dilts' code. My understanding is that he will be away on vacation soon. So see if you can catch him before that. If you miss him, one of us will dig into it.

Rao

On 07/12/2018 11:51 AM, Neil Carlson wrote:

FYI, I'm seeing this ICE with Intel 18.0.2:

|[ 92%] Building CXX object app/swarmapp/CMakeFiles/swarmapp.dir/swarmapp.cc.o /scratch/portage/portage/portage/support/basis.h(60): internal error: bad pointer jet_size={quadratic_sizes[dim], quadratic_sizes[dim]}; ^ compilation aborted for /scratch/portage/portage/app/swarmapp/swarmapp.cc (code 4) |

My cmake line:

|cmake -DENABLE_APP_TESTS=True -DCMAKE_INSTALL_PREFIX=pwd/../intel18 .. |

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/laristra/portage/issues/17, or mute the thread https://github.com/notifications/unsubscribe-auth/AQIMwAeMLHMUSYD6JH9Tig0Hg9sgLtj2ks5uF4yxgaJpZM4VNc3Z.

cferenba commented 6 years ago

@nncarlson : I ran into this same issue a while back. I committed a workaround in the bitbucket repo that apparently hasn't made it out to github yet. Give me a few minutes to track it down and I'll post it here.

nncarlson commented 6 years ago

Not a significant issue for me. I just wanted to give you guys a heads-up in case you weren't aware. Things compile fine with 17.0.6. There is an 18.0.3 out but I don't have that installed to test.

gadlanl commented 6 years ago

Neil,

I have never seen this error. I have never used intel 18.0.2. I recommend building with the Jenkins scripts. That way everything is built the way our nightly tests are built. To use Jenkins on snow (and I presume grizzly),

cd top-level-portage-directory export WORKSPACE=$PWD ./jenkins/build_matrix_entry_hpc.sh intel

You have various options here such as

./jenkins/build_matrix_entry_hpc.sh intel serial ./jenkins/build_matrix_entry_hpc.sh gcc thrust

, etc. Look in the script for the complete set.

Gary

From: Rao Garimella notifications@github.com Sent: Thursday, July 12, 2018 12:01 PM To: laristra/portage portage@noreply.github.com Cc: Subscribed subscribed@noreply.github.com Subject: Re: [laristra/portage] Internal compiler error with Intel 18.0.2 (#17)

Neil

This is something from Gary Dilts' code. My understanding is that he will be away on vacation soon. So see if you can catch him before that. If you miss him, one of us will dig into it.

Rao

On 07/12/2018 11:51 AM, Neil Carlson wrote:

FYI, I'm seeing this ICE with Intel 18.0.2:

|[ 92%] Building CXX object app/swarmapp/CMakeFiles/swarmapp.dir/swarmapp.cc.o /scratch/portage/portage/portage/support/basis.h(60): internal error: bad pointer jet_size={quadratic_sizes[dim], quadratic_sizes[dim]}; ^ compilation aborted for /scratch/portage/portage/app/swarmapp/swarmapp.cc (code 4) |

My cmake line:

|cmake -DENABLE_APP_TESTS=True -DCMAKE_INSTALL_PREFIX=pwd/../intel18 .. |

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/laristra/portage/issues/17, or mute the thread https://github.com/notifications/unsubscribe-auth/AQIMwAeMLHMUSYD6JH9Tig0Hg9sgLtj2ks5uF4yxgaJpZM4VNc3Z.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHubhttps://github.com/laristra/portage/issues/17#issuecomment-404598437, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AR0qlD7PNcI33qJsGzC21L-6niw_FKA_ks5uF47CgaJpZM4VNc3Z.

gadlanl commented 6 years ago

BTW, the code from jenkins’ script will show up in a directory called “build”.

Gary

From: Dilts, Gary Allen Sent: Thursday, July 12, 2018 12:07 PM To: 'laristra/portage' reply@reply.github.com; laristra/portage portage@noreply.github.com Cc: Subscribed subscribed@noreply.github.com Subject: RE: [laristra/portage] Internal compiler error with Intel 18.0.2 (#17)

Neil,

I have never seen this error. I have never used intel 18.0.2. I recommend building with the Jenkins scripts. That way everything is built the way our nightly tests are built. To use Jenkins on snow (and I presume grizzly),

cd top-level-portage-directory export WORKSPACE=$PWD ./jenkins/build_matrix_entry_hpc.sh intel

You have various options here such as

./jenkins/build_matrix_entry_hpc.sh intel serial ./jenkins/build_matrix_entry_hpc.sh gcc thrust

, etc. Look in the script for the complete set.

Gary

From: Rao Garimella notifications@github.com<mailto:notifications@github.com> Sent: Thursday, July 12, 2018 12:01 PM To: laristra/portage portage@noreply.github.com<mailto:portage@noreply.github.com> Cc: Subscribed subscribed@noreply.github.com<mailto:subscribed@noreply.github.com> Subject: Re: [laristra/portage] Internal compiler error with Intel 18.0.2 (#17)

Neil

This is something from Gary Dilts' code. My understanding is that he will be away on vacation soon. So see if you can catch him before that. If you miss him, one of us will dig into it.

Rao

On 07/12/2018 11:51 AM, Neil Carlson wrote:

FYI, I'm seeing this ICE with Intel 18.0.2:

|[ 92%] Building CXX object app/swarmapp/CMakeFiles/swarmapp.dir/swarmapp.cc.o /scratch/portage/portage/portage/support/basis.h(60): internal error: bad pointer jet_size={quadratic_sizes[dim], quadratic_sizes[dim]}; ^ compilation aborted for /scratch/portage/portage/app/swarmapp/swarmapp.cc (code 4) |

My cmake line:

|cmake -DENABLE_APP_TESTS=True -DCMAKE_INSTALL_PREFIX=pwd/../intel18 .. |

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/laristra/portage/issues/17, or mute the thread https://github.com/notifications/unsubscribe-auth/AQIMwAeMLHMUSYD6JH9Tig0Hg9sgLtj2ks5uF4yxgaJpZM4VNc3Z.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHubhttps://github.com/laristra/portage/issues/17#issuecomment-404598437, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AR0qlD7PNcI33qJsGzC21L-6niw_FKA_ks5uF47CgaJpZM4VNc3Z.

cferenba commented 6 years ago

FYI, the relevant section of basis.h in bitbucket master has the fix shown below. This looks like it went in shortly after we pushed v1.2.1 to github. It'll show up in our next release.

  public:
  static constexpr size_t function_size=quadratic_sizes[dim];
  // Intel 18.0.1 compiler cannot handle this declaration so we use the alternative
  //  static constexpr array<size_t,2>
  //    jet_size={quadratic_sizes[dim], quadratic_sizes[dim]};  
  static constexpr array<size_t,2>
    jet_size={function_size, function_size};
raovgarimella commented 6 years ago

Right. I forgot about this.

On 07/12/2018 12:18 PM, Charles Ferenbaugh wrote:

FYI, the relevant section of basis.h in bitbucket master has the fix shown below. This looks like it went in shortly after we pushed v1.2.1 to github. It'll show up in our next release.

|public: static constexpr size_t function_size=quadratic_sizes[dim]; // Intel 18.0.1 compiler cannot handle this declaration so we use the alternative // static constexpr array<size_t,2> // jet_size={quadratic_sizes[dim], quadratic_sizes[dim]}; static constexpr array<size_t,2> jet_size={function_size, function_size}; |

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/laristra/portage/issues/17#issuecomment-404604040, or mute the thread https://github.com/notifications/unsubscribe-auth/AQIMwOmNJK8chcTV4Kqy4piosth3b2VFks5uF5MPgaJpZM4VNc3Z.

raovgarimella commented 6 years ago

Thank you.

On 07/12/2018 12:03 PM, Neil Carlson wrote:

Not a significant issue for me. I just wanted to give you guys a heads-up in case you weren't aware. Things compile fine with 17.0.6. There is an 18.0.3 out but I don't have that installed to test.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/laristra/portage/issues/17#issuecomment-404599276, or mute the thread https://github.com/notifications/unsubscribe-auth/AQIMwCmDkckXOP-y3KtK6ApxokNYZZdcks5uF49tgaJpZM4VNc3Z.

angelaherring commented 6 years ago

Thanks for letting us know, Neal!

From: Neil Carlson notifications@github.com Sent: Thursday, July 12, 2018 12:03 PM To: laristra/portage Cc: Subscribed Subject: Re: [laristra/portage] Internal compiler error with Intel 18.0.2 (#17)

Not a significant issue for me. I just wanted to give you guys a heads-up in case you weren't aware. Things compile fine with 17.0.6. There is an 18.0.3 out but I don't have that installed to test.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHubhttps://github.com/laristra/portage/issues/17#issuecomment-404599276, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ATnRExfnToQL5bndLf4PID5fJDF9I8bAks5uF49tgaJpZM4VNc3Z.

cferenba commented 6 years ago

This fix is now included in 1.2.2 (did this a while back but forgot to close the issue, sorry...)