This is an archive of the discontinued LLVM Phabricator instance.

[libomptarget-nvptx] Enable asserts in bclib
ClosedPublic

Authored by Hahnfeld on Sep 30 2018, 2:49 AM.

Details

Summary

If the user requested LIBOMPTARGET_NVPTX_DEBUG, include asserts in
the bitcode library. Everything else will have very unpleasent
effects because asserts will appear when falling back to the static
library libomptarget-nvptx.a.

Diff Detail

Event Timeline

Hahnfeld created this revision.Sep 30 2018, 2:49 AM
grokos accepted this revision.Oct 1 2018, 1:44 AM

Looks good. I have no idea why this option was left there...

This revision is now accepted and ready to land.Oct 1 2018, 1:44 AM
This revision was automatically updated to reflect the committed changes.

Looks good. I have no idea why this option was left there...

For the record: I think that's a leftover from D45530.