[PATCH] [PM] Change the core design of the TTI analysis to use a polymorphic type erased interface and a single analysis pass rather than an extremely complex analysis group.

Chandler Carruth chandlerc at gmail.com
Fri Jan 30 19:31:55 PST 2015


On Fri, Jan 30, 2015 at 6:42 PM, hfinkel at anl.gov <hfinkel at anl.gov> wrote:

> ================
> Comment at: include/llvm/Analysis/TargetTransformInfo.h:58
> @@ -65,1 +57,3 @@
> +  /// \brief Construct a TTI object around a type which implements the
> API in
> +  /// the \c Concept below.
>    ///
> ----------------
> I understand the comment, but I think it can be made less
> potentially-confusing. How about:
>
>   Construct a TTI object using a type implementing the Concept API below.
>

FYI, done. Much better wording.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150130/a5f3d58b/attachment.html>


More information about the llvm-commits mailing list