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

Karthik Bhat kv.bhat at samsung.com
Thu Apr 16 06:30:14 PDT 2015


Hi Renato,
Thanks for your valuable inputs. I have addressed one of your comments and modified the comments as per your suggestion.
Will wait for your/James input on if to move the enums into namespace/structure.

Yes I have run make check-all on X86 and there are no regressions.

Sure I will update the LoopInterchange pass assuming this change. We wont need any additional refactoring and we can reuse these functions as it is.

Thanks once again for your time and support.

Thanks & 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.23851.patch
Type: text/x-patch
Size: 40511 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150416/c6d0c526/attachment.bin>


More information about the llvm-commits mailing list