[polly][vectorizer] do not emit bitcast i32** to <1 x i32*>*

Sebastian Pop spop at codeaurora.org
Mon Oct 7 09:48:11 PDT 2013


Tobias Grosser wrote:
> 1) Bitcast patch
> 
> The patch goes definitely in the right direction. However, it would
> be nice if you could besides modifying the existing test cases also
> provide a test cases that fail without these changes.

I will reduce a testcase and repost the patch for review.

> 
> 2) Remove -polly-cloog
> 
> By removing -polly-cloog you basically disable this test. As we
> should not use -polly-cloog in test/ScopInfo, disabling this test
> may make sense. However, then please remove the entire file.

https://llvm.org/svn/llvm-project/polly/trunk@192109

> 
> 3) Replace -polly-codegen with -polly-codegen-isl
> 
> LGTM.

https://llvm.org/svn/llvm-project/polly/trunk@192110

Thanks for your review,
Sebastian
-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
hosted by The Linux Foundation



More information about the llvm-commits mailing list