r223241 - Fix isThreadModelSupported()'s comment. NFC

Richard Smith richard at metafoo.co.uk
Wed Dec 3 19:10:38 PST 2014


FYI, the preferred fix is to replace the function name with \brief; see
http://llvm.org/docs/CodingStandards.html#doxygen-use-in-documentation-comments,
"Don’t duplicate function or class name at the beginning of the comment."

On Wed, Dec 3, 2014 at 9:04 AM, Jonathan Roelofs <jonathan at codesourcery.com>
wrote:

> Author: jroelofs
> Date: Wed Dec  3 11:04:52 2014
> New Revision: 223241
>
> URL: http://llvm.org/viewvc/llvm-project?rev=223241&view=rev
> Log:
> Fix isThreadModelSupported()'s comment. NFC
>
> Modified:
>     cfe/trunk/include/clang/Driver/ToolChain.h
>
> Modified: cfe/trunk/include/clang/Driver/ToolChain.h
> URL:
> http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Driver/ToolChain.h?rev=223241&r1=223240&r2=223241&view=diff
>
> ==============================================================================
> --- cfe/trunk/include/clang/Driver/ToolChain.h (original)
> +++ cfe/trunk/include/clang/Driver/ToolChain.h Wed Dec  3 11:04:52 2014
> @@ -251,7 +251,7 @@ public:
>    /// getThreadModel() - Which thread model does this target use?
>    virtual std::string getThreadModel() const { return "posix"; }
>
> -  /// supportsThreadModel() - Does this target support a thread model?
> +  /// isThreadModelSupported() - Does this target support a thread model?
>    virtual bool isThreadModelSupported(const StringRef Model) const;
>
>    /// ComputeLLVMTriple - Return the LLVM target triple to use, after
> taking
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20141203/0fdfe4fa/attachment.html>


More information about the cfe-commits mailing list