[PATCH] [Polly] Support for math/misc intrinsics
Tobias Grosser
phabricator at grosser.es
Sun Sep 7 23:20:53 PDT 2014
Hi Johannes,
I just tried your test case without the actual change to Polly and it already passes. The reason is most likely that all these intrinsics are side effect free functions and can consequently already be handled by Polly.
I think adding this test case is still valuable, but I would suggest a couple of minor changes to minimize the test case:
1) Call the function 'intrinsics' instead of jd
2) Use just a single loop and a single array.
There is no value added by a second loop and a second array, it just adds more code to the test case.
3) No need to call pow twice
We could possibly also remove the "intrinsics" comment (or explain that only more complex buildins such as memcpy need special handling).
Thanks,
Tobias
http://reviews.llvm.org/D5225
More information about the llvm-commits
mailing list