[PATCH] D45728: [Polly] Allow arbitrary function calls for debugging purposes.

[Michael Kruse via llvm-commits]

Could you give me a hint how to find that old revision?

Do you remember why it was kicked out?

Why do you think this version is not proper/sound?

Michael


2018-04-17 14:22 GMT-05:00 Johannes Doerfert <johannesdoerfert at gmail.com>:
> Funny thing is we actually had that functionality at some point years ago
> but kicked it out. Maybe check the old code and if you want proper/sounds
> support for those calls let me know I have a patch.
>
> On Tue, Apr 17, 2018, 20:02 Michael Kruse via Phabricator
> <reviews at reviews.llvm.org> wrote:
>>
>> Meinersbur updated this revision to Diff 142813.
>> Meinersbur added a comment.
>>
>> Use a ScopStmt's instruction list.
>>
>>
>> https://reviews.llvm.org/D45728
>>
>> Files:
>>   include/polly/Support/ScopHelper.h
>>   lib/Analysis/ScopBuilder.cpp
>>   lib/Analysis/ScopDetection.cpp
>>   lib/Analysis/ScopInfo.cpp
>>   lib/Support/ScopHelper.cpp
>>   test/ScopInfo/debug_call.ll
>>
>> --
>> You received this message because you are subscribed to the Google Groups
>> "Polly Development" group.
>> To unsubscribe from this group and stop receiving emails from it, send an
>> email to polly-dev+unsubscribe at googlegroups.com.
>> For more options, visit https://groups.google.com/d/optout.
>
> --
> You received this message because you are subscribed to the Google Groups
> "Polly Development" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to polly-dev+unsubscribe at googlegroups.com.
> For more options, visit https://groups.google.com/d/optout.