[PATCH] Refactor identification of reductions and expose them as utility functions

Karthik Bhat kv.bhat at samsung.com
Thu Apr 16 05:04:32 PDT 2015


Hi James,
Thanks for the review. Yes it makes sense to move it out of LoopVectorizer.cpp. Updated the code as per reveiw comments. 
Added a file LoopUtils.cpp in lib/Transforms/Utils with refactored code. We can add any additional common loop functionalities in this util file.

Please let me know if this looks good to you.

Thanks and Regards
Karthik Bhat


http://reviews.llvm.org/D9046

Files:
  include/llvm/Transforms/Utils/LoopUtils.h
  lib/Transforms/Utils/CMakeLists.txt
  lib/Transforms/Utils/LoopUtils.cpp
  lib/Transforms/Vectorize/LoopVectorize.cpp

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D9046.23849.patch
Type: text/x-patch
Size: 40439 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150416/b2b574dd/attachment.bin>


More information about the llvm-commits mailing list