[PATCH] D28694: Hexagon-specific loop idiom recognition

[Krzysztof Parzyszek via Phabricator via llvm-commits]

kparzysz added inline comments.


================
Comment at: include/llvm/Transforms/IPO/PassManagerBuilder.h:106
+    /// target-independent loop idiom recognition pass.
+    EP_LoopIdiom,
+
----------------
mehdi_amini wrote:
> Side note, but somehow related to the "too specific" aspect I mentioned,  I'd call it `EP_BeforeLoopIdiom` and I'd describe it:
> 
> ```
> /// This extension point allows adding passes before loop-idiom 
> /// recognition passes. It can be used for example  by a target to
> /// add a specific loop-idiom recognition passes.
> ```
> 
> 
That could certainly be done.


Repository:
  rL LLVM

https://reviews.llvm.org/D28694