This is an archive of the discontinued LLVM Phabricator instance.

[Polly][isl++] Define dump() functions in translation unit. NFC.
AbandonedPublic

Authored by Meinersbur on Sep 27 2017, 1:25 AM.

Details

Reviewers
None
Summary

"Cleaner" version of D38198. The dump() functions are non-inline defined in isl-noexceptions.cpp such there is always a symbol for it that can be called in a debugger.

This gives up isl++ being header-only and requires support by the isl++ binding generator (not part of this patch).

There is an issue if PollyISL is linked as a static library. No symbol of isl-noexceptions.o is directly used, so the compiler does not add that file when linking. It seems to work with Visual Studio anyway, but not in gcc.

Diff Detail

Event Timeline

Meinersbur created this revision.Sep 27 2017, 1:25 AM
Meinersbur abandoned this revision.Oct 31 2017, 3:31 AM

Abandon because the alternative was committed