<div dir="ltr">No technical problem, but the coding standards do say this "<tt class="" style="font-family:Consolas,'Deja Vu Sans Mono','Bitstream Vera Sans Mono',monospace;font-size:0.95em;color:rgb(0,0,0);line-height:21px"><span class="">StringRef</span></tt><span style="color:rgb(0,0,0);font-family:'Lucida Grande','Lucida Sans Unicode',Geneva,Verdana,sans-serif;font-size:14px;line-height:21px">Â is small and pervasive enough in LLVM that it should always be passed by value."</span></div>
<div class="gmail_extra"><br><br><div class="gmail_quote">On Sat, Aug 30, 2014 at 10:06 AM, Aaron Ballman <span dir="ltr"><<a href="mailto:aaron@aaronballman.com" target="_blank">aaron@aaronballman.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On Sat, Aug 30, 2014 at 12:48 PM, Craig Topper <<a href="mailto:craig.topper@gmail.com">craig.topper@gmail.com</a>> wrote:<br>
> Author: ctopper<br>
> Date: Sat Aug 30 11:48:02 2014<br>
> New Revision: 216820<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=216820&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=216820&view=rev</a><br>
> Log:<br>
> Fix some cases where StringRef was being passed by const reference. Remove const from some other StringRefs since its implicitly const already.<br>
<br>
</div>Just curious, but is there a technical problem with StringRef being<br>
passed by const reference?<br>
<span class="HOEnZb"><font color="#888888"><br>
~Aaron<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
><br>
> Modified:<br>
>Â Â Â llvm/trunk/include/llvm/MC/MCDwarf.h<br>
>Â Â Â llvm/trunk/include/llvm/MC/SubtargetFeature.h<br>
>Â Â Â llvm/trunk/include/llvm/Support/Path.h<br>
>Â Â Â llvm/trunk/include/llvm/Support/SpecialCaseList.h<br>
>Â Â Â llvm/trunk/include/llvm/Target/TargetMachine.h<br>
>Â Â Â llvm/trunk/lib/CodeGen/AsmPrinter/DIE.h<br>
>Â Â Â llvm/trunk/lib/CodeGen/AsmPrinter/DwarfUnit.h<br>
>Â Â Â llvm/trunk/lib/MC/SubtargetFeature.cpp<br>
>Â Â Â llvm/trunk/lib/Support/Path.cpp<br>
>Â Â Â llvm/trunk/lib/Support/SpecialCaseList.cpp<br>
>Â Â Â llvm/trunk/lib/Support/YAMLParser.cpp<br>
>Â Â Â llvm/trunk/lib/Target/ARM/MCTargetDesc/ARMAsmBackend.cpp<br>
>Â Â Â llvm/trunk/lib/Target/Mips/AsmParser/MipsAsmParser.cpp<br>
>Â Â Â llvm/trunk/lib/Target/Mips/MipsISelLowering.cpp<br>
>Â Â Â llvm/trunk/lib/Target/Mips/MipsISelLowering.h<br>
>Â Â Â llvm/trunk/lib/Target/Mips/MipsMachineFunction.cpp<br>
>Â Â Â llvm/trunk/lib/Target/Mips/MipsMachineFunction.h<br>
>Â Â Â llvm/trunk/lib/Target/NVPTX/MCTargetDesc/NVPTXMCAsmInfo.cpp<br>
>Â Â Â llvm/trunk/lib/Target/NVPTX/MCTargetDesc/NVPTXMCAsmInfo.h<br>
>Â Â Â llvm/trunk/lib/Transforms/Instrumentation/DataFlowSanitizer.cpp<br>
>Â Â Â llvm/trunk/tools/llvm-dwarfdump/llvm-dwarfdump.cpp<br>
><br>
> Modified: llvm/trunk/include/llvm/MC/MCDwarf.h<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/MC/MCDwarf.h?rev=216820&r1=216819&r2=216820&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/MC/MCDwarf.h?rev=216820&r1=216819&r2=216820&view=diff</a><br>
> ==============================================================================<br>
> --- llvm/trunk/include/llvm/MC/MCDwarf.h (original)<br>
> +++ llvm/trunk/include/llvm/MC/MCDwarf.h Sat Aug 30 11:48:02 2014<br>
> @@ -457,7 +457,7 @@ public:<br>
>Â Â Â return Offset;<br>
>Â Â }<br>
><br>
> -Â const StringRef getValues() const {<br>
> +Â StringRef getValues() const {<br>
>Â Â Â assert(Operation == OpEscape);<br>
>Â Â Â return StringRef(&Values[0], Values.size());<br>
>Â Â }<br>
><br>
> Modified: llvm/trunk/include/llvm/MC/SubtargetFeature.h<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/MC/SubtargetFeature.h?rev=216820&r1=216819&r2=216820&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/MC/SubtargetFeature.h?rev=216820&r1=216819&r2=216820&view=diff</a><br>
> ==============================================================================<br>
> --- llvm/trunk/include/llvm/MC/SubtargetFeature.h (original)<br>
> +++ llvm/trunk/include/llvm/MC/SubtargetFeature.h Sat Aug 30 11:48:02 2014<br>
> @@ -72,21 +72,21 @@ struct SubtargetInfoKV {<br>
>Â class SubtargetFeatures {<br>
>Â Â std::vector<std::string> Features;Â Â // Subtarget features as a vector<br>
>Â public:<br>
> -Â explicit SubtargetFeatures(const StringRef Initial = "");<br>
> +Â explicit SubtargetFeatures(StringRef Initial = "");<br>
><br>
>Â Â /// Features string accessors.<br>
>Â Â std::string getString() const;<br>
><br>
>Â Â /// Adding Features.<br>
> -Â void AddFeature(const StringRef String);<br>
> +Â void AddFeature(StringRef String);<br>
><br>
>Â Â /// ToggleFeature - Toggle a feature and returns the newly updated feature<br>
>Â Â /// bits.<br>
> -Â uint64_t ToggleFeature(uint64_t Bits, const StringRef String,<br>
> +Â uint64_t ToggleFeature(uint64_t Bits, StringRef String,<br>
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â ArrayRef<SubtargetFeatureKV> FeatureTable);<br>
><br>
>Â Â /// Get feature bits of a CPU.<br>
> -Â uint64_t getFeatureBits(const StringRef CPU,<br>
> +Â uint64_t getFeatureBits(StringRef CPU,<br>
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â ArrayRef<SubtargetFeatureKV> CPUTable,<br>
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â ArrayRef<SubtargetFeatureKV> FeatureTable);<br>
><br>
><br>
> Modified: llvm/trunk/include/llvm/Support/Path.h<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Support/Path.h?rev=216820&r1=216819&r2=216820&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Support/Path.h?rev=216820&r1=216819&r2=216820&view=diff</a><br>
> ==============================================================================<br>
> --- llvm/trunk/include/llvm/Support/Path.h (original)<br>
> +++ llvm/trunk/include/llvm/Support/Path.h Sat Aug 30 11:48:02 2014<br>
> @@ -204,7 +204,7 @@ void native(SmallVectorImpl<char> &path)<br>
>Â ///<br>
>Â /// @param path Input path.<br>
>Â /// @result The root name of \a path if it has one, otherwise "".<br>
> -const StringRef root_name(StringRef path);<br>
> +StringRef root_name(StringRef path);<br>
><br>
>Â /// @brief Get root directory.<br>
>Â ///<br>
> @@ -217,7 +217,7 @@ const StringRef root_name(StringRef path<br>
>Â /// @param path Input path.<br>
>Â /// @result The root directory of \a path if it has one, otherwise<br>
>Â ///Â Â Â Â Â Â Â Â "".<br>
> -const StringRef root_directory(StringRef path);<br>
> +StringRef root_directory(StringRef path);<br>
><br>
>Â /// @brief Get root path.<br>
>Â ///<br>
> @@ -225,7 +225,7 @@ const StringRef root_directory(StringRef<br>
>Â ///<br>
>Â /// @param path Input path.<br>
>Â /// @result The root path of \a path if it has one, otherwise "".<br>
> -const StringRef root_path(StringRef path);<br>
> +StringRef root_path(StringRef path);<br>
><br>
>Â /// @brief Get relative path.<br>
>Â ///<br>
> @@ -237,7 +237,7 @@ const StringRef root_path(StringRef path<br>
>Â ///<br>
>Â /// @param path Input path.<br>
>Â /// @result The path starting after root_path if one exists, otherwise "".<br>
> -const StringRef relative_path(StringRef path);<br>
> +StringRef relative_path(StringRef path);<br>
><br>
>Â /// @brief Get parent path.<br>
>Â ///<br>
> @@ -249,7 +249,7 @@ const StringRef relative_path(StringRef<br>
>Â ///<br>
>Â /// @param path Input path.<br>
>Â /// @result The parent path of \a path if one exists, otherwise "".<br>
> -const StringRef parent_path(StringRef path);<br>
> +StringRef parent_path(StringRef path);<br>
><br>
>Â /// @brief Get filename.<br>
>Â ///<br>
> @@ -263,7 +263,7 @@ const StringRef parent_path(StringRef pa<br>
>Â /// @param path Input path.<br>
>Â /// @result The filename part of \a path. This is defined as the last component<br>
>Â ///Â Â Â Â Â of \a path.<br>
> -const StringRef filename(StringRef path);<br>
> +StringRef filename(StringRef path);<br>
><br>
>Â /// @brief Get stem.<br>
>Â ///<br>
> @@ -281,7 +281,7 @@ const StringRef filename(StringRef path)<br>
>Â ///<br>
>Â /// @param path Input path.<br>
>Â /// @result The stem of \a path.<br>
> -const StringRef stem(StringRef path);<br>
> +StringRef stem(StringRef path);<br>
><br>
>Â /// @brief Get extension.<br>
>Â ///<br>
> @@ -297,7 +297,7 @@ const StringRef stem(StringRef path);<br>
>Â ///<br>
>Â /// @param path Input path.<br>
>Â /// @result The extension of \a path.<br>
> -const StringRef extension(StringRef path);<br>
> +StringRef extension(StringRef path);<br>
><br>
>Â /// @brief Check whether the given char is a path separator on the host OS.<br>
>Â ///<br>
> @@ -308,7 +308,7 @@ bool is_separator(char value);<br>
>Â /// @brief Return the preferred separator for this platform.<br>
>Â ///<br>
>Â /// @result StringRef of the preferred separator, null-terminated.<br>
> -const StringRef get_separator();<br>
> +StringRef get_separator();<br>
><br>
>Â /// @brief Get the typical temporary directory for the system, e.g.,<br>
>Â /// "/var/tmp" or "C:/TEMP"<br>
><br>
> Modified: llvm/trunk/include/llvm/Support/SpecialCaseList.h<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Support/SpecialCaseList.h?rev=216820&r1=216819&r2=216820&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Support/SpecialCaseList.h?rev=216820&r1=216819&r2=216820&view=diff</a><br>
> ==============================================================================<br>
> --- llvm/trunk/include/llvm/Support/SpecialCaseList.h (original)<br>
> +++ llvm/trunk/include/llvm/Support/SpecialCaseList.h Sat Aug 30 11:48:02 2014<br>
> @@ -60,13 +60,13 @@ class SpecialCaseList {<br>
>Â Â /// Parses the special case list from a file. If Path is empty, returns<br>
>Â Â /// an empty special case list. On failure, returns 0 and writes an error<br>
>Â Â /// message to string.<br>
> -Â static SpecialCaseList *create(const StringRef Path, std::string &Error);<br>
> +Â static SpecialCaseList *create(StringRef Path, std::string &Error);<br>
>Â Â /// Parses the special case list from a memory buffer. On failure, returns<br>
>Â Â /// 0 and writes an error message to string.<br>
>Â Â static SpecialCaseList *create(const MemoryBuffer *MB, std::string &Error);<br>
>Â Â /// Parses the special case list from a file. On failure, reports a fatal<br>
>Â Â /// error.<br>
> -Â static SpecialCaseList *createOrDie(const StringRef Path);<br>
> +Â static SpecialCaseList *createOrDie(StringRef Path);<br>
><br>
>Â Â ~SpecialCaseList();<br>
><br>
> @@ -75,8 +75,8 @@ class SpecialCaseList {<br>
>Â Â ///Â Â @Section:<E>=@Category<br>
>Â Â /// \endcode<br>
>Â Â /// and @Query satisfies a wildcard expression <E>.<br>
> -Â bool inSection(const StringRef Section, const StringRef Query,<br>
> -Â Â Â Â Â Â Â Â Â const StringRef Category = StringRef()) const;<br>
> +Â bool inSection(StringRef Section, StringRef Query,<br>
> +Â Â Â Â Â Â Â Â Â StringRef Category = StringRef()) const;<br>
><br>
>Â Â private:<br>
>Â Â SpecialCaseList(SpecialCaseList const &) LLVM_DELETED_FUNCTION;<br>
><br>
> Modified: llvm/trunk/include/llvm/Target/TargetMachine.h<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Target/TargetMachine.h?rev=216820&r1=216819&r2=216820&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Target/TargetMachine.h?rev=216820&r1=216819&r2=216820&view=diff</a><br>
> ==============================================================================<br>
> --- llvm/trunk/include/llvm/Target/TargetMachine.h (original)<br>
> +++ llvm/trunk/include/llvm/Target/TargetMachine.h Sat Aug 30 11:48:02 2014<br>
> @@ -92,9 +92,9 @@ public:<br>
><br>
>Â Â const Target &getTarget() const { return TheTarget; }<br>
><br>
> -Â const StringRef getTargetTriple() const { return TargetTriple; }<br>
> -Â const StringRef getTargetCPU() const { return TargetCPU; }<br>
> -Â const StringRef getTargetFeatureString() const { return TargetFS; }<br>
> +Â StringRef getTargetTriple() const { return TargetTriple; }<br>
> +Â StringRef getTargetCPU() const { return TargetCPU; }<br>
> +Â StringRef getTargetFeatureString() const { return TargetFS; }<br>
><br>
>Â Â /// getSubtargetImpl - virtual method implemented by subclasses that returns<br>
>Â Â /// a reference to that target's TargetSubtargetInfo-derived member variable.<br>
><br>
> Modified: llvm/trunk/lib/CodeGen/AsmPrinter/DIE.h<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/AsmPrinter/DIE.h?rev=216820&r1=216819&r2=216820&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/AsmPrinter/DIE.h?rev=216820&r1=216819&r2=216820&view=diff</a><br>
> ==============================================================================<br>
> --- llvm/trunk/lib/CodeGen/AsmPrinter/DIE.h (original)<br>
> +++ llvm/trunk/lib/CodeGen/AsmPrinter/DIE.h Sat Aug 30 11:48:02 2014<br>
> @@ -381,10 +381,10 @@ public:<br>
>Â ///<br>
>Â class DIEString : public DIEValue {<br>
>Â Â const DIEValue *Access;<br>
> -Â const StringRef Str;<br>
> +Â StringRef Str;<br>
><br>
>Â public:<br>
> -Â DIEString(const DIEValue *Acc, const StringRef S)<br>
> +Â DIEString(const DIEValue *Acc, StringRef S)<br>
>Â Â Â Â : DIEValue(isString), Access(Acc), Str(S) {}<br>
><br>
>Â Â /// getString - Grab the string out of the object.<br>
><br>
> Modified: llvm/trunk/lib/CodeGen/AsmPrinter/DwarfUnit.h<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/AsmPrinter/DwarfUnit.h?rev=216820&r1=216819&r2=216820&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/AsmPrinter/DwarfUnit.h?rev=216820&r1=216819&r2=216820&view=diff</a><br>
> ==============================================================================<br>
> --- llvm/trunk/lib/CodeGen/AsmPrinter/DwarfUnit.h (original)<br>
> +++ llvm/trunk/lib/CodeGen/AsmPrinter/DwarfUnit.h Sat Aug 30 11:48:02 2014<br>
> @@ -275,11 +275,11 @@ public:<br>
>Â Â void addSInt(DIELoc &Die, Optional<dwarf::Form> Form, int64_t Integer);<br>
><br>
>Â Â /// addString - Add a string attribute data and value.<br>
> -Â void addString(DIE &Die, dwarf::Attribute Attribute, const StringRef Str);<br>
> +Â void addString(DIE &Die, dwarf::Attribute Attribute, StringRef Str);<br>
><br>
>Â Â /// addLocalString - Add a string attribute data and value.<br>
>Â Â void addLocalString(DIE &Die, dwarf::Attribute Attribute,<br>
> -Â Â Â Â Â Â Â Â Â Â Â const StringRef Str);<br>
> +Â Â Â Â Â Â Â Â Â Â Â StringRef Str);<br>
><br>
>Â Â /// addExpr - Add a Dwarf expression attribute data and value.<br>
>Â Â void addExpr(DIELoc &Die, dwarf::Form Form, const MCExpr *Expr);<br>
><br>
> Modified: llvm/trunk/lib/MC/SubtargetFeature.cpp<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/MC/SubtargetFeature.cpp?rev=216820&r1=216819&r2=216820&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/MC/SubtargetFeature.cpp?rev=216820&r1=216819&r2=216820&view=diff</a><br>
> ==============================================================================<br>
> --- llvm/trunk/lib/MC/SubtargetFeature.cpp (original)<br>
> +++ llvm/trunk/lib/MC/SubtargetFeature.cpp Sat Aug 30 11:48:02 2014<br>
> @@ -27,7 +27,7 @@ using namespace llvm;<br>
><br>
>Â /// hasFlag - Determine if a feature has a flag; '+' or '-'<br>
>Â ///<br>
> -static inline bool hasFlag(const StringRef Feature) {<br>
> +static inline bool hasFlag(StringRef Feature) {<br>
>Â Â assert(!Feature.empty() && "Empty string");<br>
>Â Â // Get first character<br>
>Â Â char Ch = Feature[0];<br>
> @@ -37,13 +37,13 @@ static inline bool hasFlag(const StringR<br>
><br>
>Â /// StripFlag - Return string stripped of flag.<br>
>Â ///<br>
> -static inline std::string StripFlag(const StringRef Feature) {<br>
> +static inline std::string StripFlag(StringRef Feature) {<br>
>Â Â return hasFlag(Feature) ? Feature.substr(1) : Feature;<br>
>Â }<br>
><br>
>Â /// isEnabled - Return true if enable flag; '+'.<br>
>Â ///<br>
> -static inline bool isEnabled(const StringRef Feature) {<br>
> +static inline bool isEnabled(StringRef Feature) {<br>
>Â Â assert(!Feature.empty() && "Empty string");<br>
>Â Â // Get first character<br>
>Â Â char Ch = Feature[0];<br>
> @@ -53,7 +53,7 @@ static inline bool isEnabled(const Strin<br>
><br>
>Â /// Split - Splits a string of comma separated items in to a vector of strings.<br>
>Â ///<br>
> -static void Split(std::vector<std::string> &V, const StringRef S) {<br>
> +static void Split(std::vector<std::string> &V, StringRef S) {<br>
>Â Â SmallVector<StringRef, 3> Tmp;<br>
>Â Â S.split(Tmp, ",", -1, false /* KeepEmpty */);<br>
>Â Â V.assign(Tmp.begin(), Tmp.end());<br>
> @@ -81,7 +81,7 @@ static std::string Join(const std::vecto<br>
>Â }<br>
><br>
>Â /// Adding features.<br>
> -void SubtargetFeatures::AddFeature(const StringRef String) {<br>
> +void SubtargetFeatures::AddFeature(StringRef String) {<br>
>Â Â // Don't add empty features or features we already have.<br>
>Â Â if (!String.empty())<br>
>Â Â Â // Convert to lowercase, prepend flag if we don't already have a flag.<br>
> @@ -136,7 +136,7 @@ static void Help(ArrayRef<SubtargetFeatu<br>
>Â //Â Â Â Â Â Â Â Â Â Â SubtargetFeatures Implementation<br>
>Â //===----------------------------------------------------------------------===//<br>
><br>
> -SubtargetFeatures::SubtargetFeatures(const StringRef Initial) {<br>
> +SubtargetFeatures::SubtargetFeatures(StringRef Initial) {<br>
>Â Â // Break up string into separate features<br>
>Â Â Split(Features, Initial);<br>
>Â }<br>
> @@ -181,7 +181,7 @@ void ClearImpliedBits(uint64_t &Bits, co<br>
>Â /// ToggleFeature - Toggle a feature and returns the newly updated feature<br>
>Â /// bits.<br>
>Â uint64_t<br>
> -SubtargetFeatures::ToggleFeature(uint64_t Bits, const StringRef Feature,<br>
> +SubtargetFeatures::ToggleFeature(uint64_t Bits, StringRef Feature,<br>
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â ArrayRef<SubtargetFeatureKV> FeatureTable) {<br>
><br>
>Â Â // Find feature in table.<br>
> @@ -213,7 +213,7 @@ SubtargetFeatures::ToggleFeature(uint64_<br>
>Â /// getFeatureBits - Get feature bits a CPU.<br>
>Â ///<br>
>Â uint64_t<br>
> -SubtargetFeatures::getFeatureBits(const StringRef CPU,<br>
> +SubtargetFeatures::getFeatureBits(StringRef CPU,<br>
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â ArrayRef<SubtargetFeatureKV> CPUTable,<br>
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â ArrayRef<SubtargetFeatureKV> FeatureTable) {<br>
><br>
><br>
> Modified: llvm/trunk/lib/Support/Path.cpp<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Support/Path.cpp?rev=216820&r1=216819&r2=216820&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Support/Path.cpp?rev=216820&r1=216819&r2=216820&view=diff</a><br>
> ==============================================================================<br>
> --- llvm/trunk/lib/Support/Path.cpp (original)<br>
> +++ llvm/trunk/lib/Support/Path.cpp Sat Aug 30 11:48:02 2014<br>
> @@ -359,7 +359,7 @@ bool reverse_iterator::operator==(const<br>
>Â Â Â Â Â Â Position == RHS.Position;<br>
>Â }<br>
><br>
> -const StringRef root_path(StringRef path) {<br>
> +StringRef root_path(StringRef path) {<br>
>Â Â const_iterator b = begin(path),<br>
>Â Â Â Â Â Â Â Â Â Â pos = b,<br>
>Â Â Â Â Â Â Â Â Â Â e = end(path);<br>
> @@ -391,7 +391,7 @@ const StringRef root_path(StringRef path<br>
>Â Â return StringRef();<br>
>Â }<br>
><br>
> -const StringRef root_name(StringRef path) {<br>
> +StringRef root_name(StringRef path) {<br>
>Â Â const_iterator b = begin(path),<br>
>Â Â Â Â Â Â Â Â Â Â e = end(path);<br>
>Â Â if (b != e) {<br>
> @@ -413,7 +413,7 @@ const StringRef root_name(StringRef path<br>
>Â Â return StringRef();<br>
>Â }<br>
><br>
> -const StringRef root_directory(StringRef path) {<br>
> +StringRef root_directory(StringRef path) {<br>
>Â Â const_iterator b = begin(path),<br>
>Â Â Â Â Â Â Â Â Â Â pos = b,<br>
>Â Â Â Â Â Â Â Â Â Â e = end(path);<br>
> @@ -442,7 +442,7 @@ const StringRef root_directory(StringRef<br>
>Â Â return StringRef();<br>
>Â }<br>
><br>
> -const StringRef relative_path(StringRef path) {<br>
> +StringRef relative_path(StringRef path) {<br>
>Â Â StringRef root = root_path(path);<br>
>Â Â return path.substr(root.size());<br>
>Â }<br>
> @@ -494,7 +494,7 @@ void append(SmallVectorImpl<char> &path,<br>
>Â Â Â path::append(path, *begin);<br>
>Â }<br>
><br>
> -const StringRef parent_path(StringRef path) {<br>
> +StringRef parent_path(StringRef path) {<br>
>Â Â size_t end_pos = parent_path_end(path);<br>
>Â Â if (end_pos == StringRef::npos)<br>
>Â Â Â return StringRef();<br>
> @@ -552,11 +552,11 @@ void native(SmallVectorImpl<char> &Path)<br>
>Â #endif<br>
>Â }<br>
><br>
> -const StringRef filename(StringRef path) {<br>
> +StringRef filename(StringRef path) {<br>
>Â Â return *rbegin(path);<br>
>Â }<br>
><br>
> -const StringRef stem(StringRef path) {<br>
> +StringRef stem(StringRef path) {<br>
>Â Â StringRef fname = filename(path);<br>
>Â Â size_t pos = fname.find_last_of('.');<br>
>Â Â if (pos == StringRef::npos)<br>
> @@ -569,7 +569,7 @@ const StringRef stem(StringRef path) {<br>
>Â Â Â Â return fname.substr(0, pos);<br>
>Â }<br>
><br>
> -const StringRef extension(StringRef path) {<br>
> +StringRef extension(StringRef path) {<br>
>Â Â StringRef fname = filename(path);<br>
>Â Â size_t pos = fname.find_last_of('.');<br>
>Â Â if (pos == StringRef::npos)<br>
> @@ -594,7 +594,7 @@ bool is_separator(char value) {<br>
><br>
>Â static const char preferred_separator_string[] = { preferred_separator, '\0' };<br>
><br>
> -const StringRef get_separator() {<br>
> +StringRef get_separator() {<br>
>Â Â return preferred_separator_string;<br>
>Â }<br>
><br>
><br>
> Modified: llvm/trunk/lib/Support/SpecialCaseList.cpp<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Support/SpecialCaseList.cpp?rev=216820&r1=216819&r2=216820&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Support/SpecialCaseList.cpp?rev=216820&r1=216819&r2=216820&view=diff</a><br>
> ==============================================================================<br>
> --- llvm/trunk/lib/Support/SpecialCaseList.cpp (original)<br>
> +++ llvm/trunk/lib/Support/SpecialCaseList.cpp Sat Aug 30 11:48:02 2014<br>
> @@ -48,8 +48,7 @@ struct SpecialCaseList::Entry {<br>
><br>
>Â SpecialCaseList::SpecialCaseList() : Entries() {}<br>
><br>
> -SpecialCaseList *SpecialCaseList::create(<br>
> -Â Â const StringRef Path, std::string &Error) {<br>
> +SpecialCaseList *SpecialCaseList::create(StringRef Path, std::string &Error) {<br>
>Â Â if (Path.empty())<br>
>Â Â Â return new SpecialCaseList();<br>
>Â Â ErrorOr<std::unique_ptr<MemoryBuffer>> FileOrErr =<br>
> @@ -69,7 +68,7 @@ SpecialCaseList *SpecialCaseList::create<br>
>Â Â return SCL.release();<br>
>Â }<br>
><br>
> -SpecialCaseList *SpecialCaseList::createOrDie(const StringRef Path) {<br>
> +SpecialCaseList *SpecialCaseList::createOrDie(StringRef Path) {<br>
>Â Â std::string Error;<br>
>Â Â if (SpecialCaseList *SCL = create(Path, Error))<br>
>Â Â Â return SCL;<br>
> @@ -157,8 +156,8 @@ bool SpecialCaseList::parse(const Memory<br>
><br>
>Â SpecialCaseList::~SpecialCaseList() {}<br>
><br>
> -bool SpecialCaseList::inSection(const StringRef Section, const StringRef Query,<br>
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â const StringRef Category) const {<br>
> +bool SpecialCaseList::inSection(StringRef Section, StringRef Query,<br>
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â StringRef Category) const {<br>
>Â Â StringMap<StringMap<Entry> >::const_iterator I = Entries.find(Section);<br>
>Â Â if (I == Entries.end()) return false;<br>
>Â Â StringMap<Entry>::const_iterator II = I->second.find(Category);<br>
><br>
> Modified: llvm/trunk/lib/Support/YAMLParser.cpp<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Support/YAMLParser.cpp?rev=216820&r1=216819&r2=216820&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Support/YAMLParser.cpp?rev=216820&r1=216819&r2=216820&view=diff</a><br>
> ==============================================================================<br>
> --- llvm/trunk/lib/Support/YAMLParser.cpp (original)<br>
> +++ llvm/trunk/lib/Support/YAMLParser.cpp Sat Aug 30 11:48:02 2014<br>
> @@ -259,7 +259,7 @@ namespace yaml {<br>
>Â /// @brief Scans YAML tokens from a MemoryBuffer.<br>
>Â class Scanner {<br>
>Â public:<br>
> -Â Scanner(const StringRef Input, SourceMgr &SM);<br>
> +Â Scanner(StringRef Input, SourceMgr &SM);<br>
>Â Â Scanner(MemoryBufferRef Buffer, SourceMgr &SM_);<br>
><br>
>Â Â /// @brief Parse the next token and return it without popping it.<br>
><br>
> Modified: llvm/trunk/lib/Target/ARM/MCTargetDesc/ARMAsmBackend.cpp<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/ARM/MCTargetDesc/ARMAsmBackend.cpp?rev=216820&r1=216819&r2=216820&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/ARM/MCTargetDesc/ARMAsmBackend.cpp?rev=216820&r1=216819&r2=216820&view=diff</a><br>
> ==============================================================================<br>
> --- llvm/trunk/lib/Target/ARM/MCTargetDesc/ARMAsmBackend.cpp (original)<br>
> +++ llvm/trunk/lib/Target/ARM/MCTargetDesc/ARMAsmBackend.cpp Sat Aug 30 11:48:02 2014<br>
> @@ -44,7 +44,7 @@ class ARMAsmBackend : public MCAsmBacken<br>
>Â Â bool isThumbMode;Â Â Â // Currently emitting Thumb code.<br>
>Â Â bool IsLittleEndian;Â // Big or little endian.<br>
>Â public:<br>
> -Â ARMAsmBackend(const Target &T, const StringRef TT, bool IsLittle)<br>
> +Â ARMAsmBackend(const Target &T, StringRef TT, bool IsLittle)<br>
>Â Â Â : MCAsmBackend(), STI(ARM_MC::createARMMCSubtargetInfo(TT, "", "")),<br>
>Â Â Â Â isThumbMode(TT.startswith("thumb")), IsLittleEndian(IsLittle) {}<br>
><br>
> @@ -761,7 +761,7 @@ namespace {<br>
>Â // FIXME: This should be in a separate file.<br>
>Â class ARMWinCOFFAsmBackend : public ARMAsmBackend {<br>
>Â public:<br>
> -Â ARMWinCOFFAsmBackend(const Target &T, const StringRef &Triple)<br>
> +Â ARMWinCOFFAsmBackend(const Target &T, StringRef Triple)<br>
>Â Â Â : ARMAsmBackend(T, Triple, true) { }<br>
>Â Â MCObjectWriter *createObjectWriter(raw_ostream &OS) const override {<br>
>Â Â Â return createARMWinCOFFObjectWriter(OS, /*Is64Bit=*/false);<br>
> @@ -773,7 +773,7 @@ public:<br>
>Â class ELFARMAsmBackend : public ARMAsmBackend {<br>
>Â public:<br>
>Â Â uint8_t OSABI;<br>
> -Â ELFARMAsmBackend(const Target &T, const StringRef TT,<br>
> +Â ELFARMAsmBackend(const Target &T, StringRef TT,<br>
>Â Â Â Â Â Â Â Â Â Â Â uint8_t OSABI, bool IsLittle)<br>
>Â Â Â : ARMAsmBackend(T, TT, IsLittle), OSABI(OSABI) { }<br>
><br>
> @@ -786,7 +786,7 @@ public:<br>
>Â class DarwinARMAsmBackend : public ARMAsmBackend {<br>
>Â public:<br>
>Â Â const MachO::CPUSubTypeARM Subtype;<br>
> -Â DarwinARMAsmBackend(const Target &T, const StringRef TT,<br>
> +Â DarwinARMAsmBackend(const Target &T, StringRef TT,<br>
>Â Â Â Â Â Â Â Â Â Â Â Â MachO::CPUSubTypeARM st)<br>
>Â Â Â : ARMAsmBackend(T, TT, /* IsLittleEndian */ true), Subtype(st) {<br>
>Â Â Â Â HasDataInCodeSupport = true;<br>
><br>
> Modified: llvm/trunk/lib/Target/Mips/AsmParser/MipsAsmParser.cpp<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Mips/AsmParser/MipsAsmParser.cpp?rev=216820&r1=216819&r2=216820&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Mips/AsmParser/MipsAsmParser.cpp?rev=216820&r1=216819&r2=216820&view=diff</a><br>
> ==============================================================================<br>
> --- llvm/trunk/lib/Target/Mips/AsmParser/MipsAsmParser.cpp (original)<br>
> +++ llvm/trunk/lib/Target/Mips/AsmParser/MipsAsmParser.cpp Sat Aug 30 11:48:02 2014<br>
> @@ -2081,7 +2081,7 @@ bool MipsAsmParser::searchSymbolAlias(Op<br>
>Â Â Â Â return false;<br>
>Â Â Â if (Expr->getKind() == MCExpr::SymbolRef) {<br>
>Â Â Â Â const MCSymbolRefExpr *Ref = static_cast<const MCSymbolRefExpr *>(Expr);<br>
> -Â Â Â const StringRef DefSymbol = Ref->getSymbol().getName();<br>
> +Â Â Â StringRef DefSymbol = Ref->getSymbol().getName();<br>
>Â Â Â Â if (DefSymbol.startswith("$")) {<br>
>Â Â Â Â Â OperandMatchResultTy ResTy =<br>
>Â Â Â Â Â Â Â MatchAnyRegisterNameWithoutDollar(Operands, DefSymbol.substr(1), S);<br>
><br>
> Modified: llvm/trunk/lib/Target/Mips/MipsISelLowering.cpp<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Mips/MipsISelLowering.cpp?rev=216820&r1=216819&r2=216820&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Mips/MipsISelLowering.cpp?rev=216820&r1=216819&r2=216820&view=diff</a><br>
> ==============================================================================<br>
> --- llvm/trunk/lib/Target/Mips/MipsISelLowering.cpp (original)<br>
> +++ llvm/trunk/lib/Target/Mips/MipsISelLowering.cpp Sat Aug 30 11:48:02 2014<br>
> @@ -3034,7 +3034,7 @@ MipsTargetLowering::getSingleConstraintM<br>
>Â /// that is returned indicates whether parsing was successful. The second flag<br>
>Â /// is true if the numeric part exists.<br>
>Â static std::pair<bool, bool><br>
> -parsePhysicalReg(const StringRef &C, std::string &Prefix,<br>
> +parsePhysicalReg(StringRef C, std::string &Prefix,<br>
>Â Â Â Â Â Â Â Â Â Â unsigned long long &Reg) {<br>
>Â Â if (C.front() != '{' || C.back() != '}')<br>
>Â Â Â return std::make_pair(false, false);<br>
> @@ -3055,7 +3055,7 @@ parsePhysicalReg(const StringRef &C, std<br>
>Â }<br>
><br>
>Â std::pair<unsigned, const TargetRegisterClass *> MipsTargetLowering::<br>
> -parseRegForInlineAsmConstraint(const StringRef &C, MVT VT) const {<br>
> +parseRegForInlineAsmConstraint(StringRef C, MVT VT) const {<br>
>Â Â const TargetRegisterInfo *TRI =<br>
>Â Â Â Â getTargetMachine().getSubtargetImpl()->getRegisterInfo();<br>
>Â Â const TargetRegisterClass *RC;<br>
><br>
> Modified: llvm/trunk/lib/Target/Mips/MipsISelLowering.h<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Mips/MipsISelLowering.h?rev=216820&r1=216819&r2=216820&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Mips/MipsISelLowering.h?rev=216820&r1=216819&r2=216820&view=diff</a><br>
> ==============================================================================<br>
> --- llvm/trunk/lib/Target/Mips/MipsISelLowering.h (original)<br>
> +++ llvm/trunk/lib/Target/Mips/MipsISelLowering.h Sat Aug 30 11:48:02 2014<br>
> @@ -563,7 +563,7 @@ namespace llvm {<br>
>Â Â Â /// This function parses registers that appear in inline-asm constraints.<br>
>Â Â Â /// It returns pair (0, 0) on failure.<br>
>Â Â Â std::pair<unsigned, const TargetRegisterClass *><br>
> -Â Â parseRegForInlineAsmConstraint(const StringRef &C, MVT VT) const;<br>
> +Â Â parseRegForInlineAsmConstraint(StringRef C, MVT VT) const;<br>
><br>
>Â Â Â std::pair<unsigned, const TargetRegisterClass*><br>
>Â Â Â Â Â Â Â Â getRegForInlineAsmConstraint(const std::string &Constraint,<br>
><br>
> Modified: llvm/trunk/lib/Target/Mips/MipsMachineFunction.cpp<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Mips/MipsMachineFunction.cpp?rev=216820&r1=216819&r2=216820&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Mips/MipsMachineFunction.cpp?rev=216820&r1=216819&r2=216820&view=diff</a><br>
> ==============================================================================<br>
> --- llvm/trunk/lib/Target/Mips/MipsMachineFunction.cpp (original)<br>
> +++ llvm/trunk/lib/Target/Mips/MipsMachineFunction.cpp Sat Aug 30 11:48:02 2014<br>
> @@ -24,7 +24,7 @@ FixGlobalBaseReg("mips-fix-global-base-r<br>
>Â Â Â Â Â Â Â Â Â Â cl::desc("Always use $gp as the global base register."));<br>
><br>
>Â // class MipsCallEntry.<br>
> -MipsCallEntry::MipsCallEntry(const StringRef &N) {<br>
> +MipsCallEntry::MipsCallEntry(StringRef N) {<br>
>Â #ifndef NDEBUG<br>
>Â Â Name = N;<br>
>Â Â Val = nullptr;<br>
> @@ -119,7 +119,7 @@ bool MipsFunctionInfo::isEhDataRegFI(int<br>
>Â Â Â Â Â Â Â Â Â Â Â Â Â || FI == EhDataRegFI[2] || FI == EhDataRegFI[3]);<br>
>Â }<br>
><br>
> -MachinePointerInfo MipsFunctionInfo::callPtrInfo(const StringRef &Name) {<br>
> +MachinePointerInfo MipsFunctionInfo::callPtrInfo(StringRef Name) {<br>
>Â Â const MipsCallEntry *&E = ExternalCallEntries[Name];<br>
><br>
>Â Â if (!E)<br>
><br>
> Modified: llvm/trunk/lib/Target/Mips/MipsMachineFunction.h<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Mips/MipsMachineFunction.h?rev=216820&r1=216819&r2=216820&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Mips/MipsMachineFunction.h?rev=216820&r1=216819&r2=216820&view=diff</a><br>
> ==============================================================================<br>
> --- llvm/trunk/lib/Target/Mips/MipsMachineFunction.h (original)<br>
> +++ llvm/trunk/lib/Target/Mips/MipsMachineFunction.h Sat Aug 30 11:48:02 2014<br>
> @@ -34,7 +34,7 @@ namespace llvm {<br>
>Â /// resolved by lazy-binding.<br>
>Â class MipsCallEntry : public PseudoSourceValue {<br>
>Â public:<br>
> -Â explicit MipsCallEntry(const StringRef &N);<br>
> +Â explicit MipsCallEntry(StringRef N);<br>
>Â Â explicit MipsCallEntry(const GlobalValue *V);<br>
>Â Â bool isConstant(const MachineFrameInfo *) const override;<br>
>Â Â bool isAliased(const MachineFrameInfo *) const override;<br>
> @@ -88,7 +88,7 @@ public:<br>
><br>
>Â Â /// \brief Create a MachinePointerInfo that has a MipsCallEntr object<br>
>Â Â /// representing a GOT entry for an external function.<br>
> -Â MachinePointerInfo callPtrInfo(const StringRef &Name);<br>
> +Â MachinePointerInfo callPtrInfo(StringRef Name);<br>
><br>
>Â Â /// \brief Create a MachinePointerInfo that has a MipsCallEntr object<br>
>Â Â /// representing a GOT entry for a global function.<br>
><br>
> Modified: llvm/trunk/lib/Target/NVPTX/MCTargetDesc/NVPTXMCAsmInfo.cpp<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/NVPTX/MCTargetDesc/NVPTXMCAsmInfo.cpp?rev=216820&r1=216819&r2=216820&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/NVPTX/MCTargetDesc/NVPTXMCAsmInfo.cpp?rev=216820&r1=216819&r2=216820&view=diff</a><br>
> ==============================================================================<br>
> --- llvm/trunk/lib/Target/NVPTX/MCTargetDesc/NVPTXMCAsmInfo.cpp (original)<br>
> +++ llvm/trunk/lib/Target/NVPTX/MCTargetDesc/NVPTXMCAsmInfo.cpp Sat Aug 30 11:48:02 2014<br>
> @@ -25,7 +25,7 @@ static cl::opt<bool> CompileForDebugging<br>
><br>
>Â void NVPTXMCAsmInfo::anchor() {}<br>
><br>
> -NVPTXMCAsmInfo::NVPTXMCAsmInfo(const StringRef &TT) {<br>
> +NVPTXMCAsmInfo::NVPTXMCAsmInfo(StringRef TT) {<br>
>Â Â Triple TheTriple(TT);<br>
>Â Â if (TheTriple.getArch() == Triple::nvptx64) {<br>
>Â Â Â PointerSize = CalleeSaveStackSlotSize = 8;<br>
><br>
> Modified: llvm/trunk/lib/Target/NVPTX/MCTargetDesc/NVPTXMCAsmInfo.h<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/NVPTX/MCTargetDesc/NVPTXMCAsmInfo.h?rev=216820&r1=216819&r2=216820&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/NVPTX/MCTargetDesc/NVPTXMCAsmInfo.h?rev=216820&r1=216819&r2=216820&view=diff</a><br>
> ==============================================================================<br>
> --- llvm/trunk/lib/Target/NVPTX/MCTargetDesc/NVPTXMCAsmInfo.h (original)<br>
> +++ llvm/trunk/lib/Target/NVPTX/MCTargetDesc/NVPTXMCAsmInfo.h Sat Aug 30 11:48:02 2014<br>
> @@ -23,7 +23,7 @@ class StringRef;<br>
>Â class NVPTXMCAsmInfo : public MCAsmInfo {<br>
>Â Â virtual void anchor();<br>
>Â public:<br>
> -Â explicit NVPTXMCAsmInfo(const StringRef &TT);<br>
> +Â explicit NVPTXMCAsmInfo(StringRef TT);<br>
>Â };<br>
>Â } // namespace llvm<br>
><br>
><br>
> Modified: llvm/trunk/lib/Transforms/Instrumentation/DataFlowSanitizer.cpp<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Instrumentation/DataFlowSanitizer.cpp?rev=216820&r1=216819&r2=216820&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Instrumentation/DataFlowSanitizer.cpp?rev=216820&r1=216819&r2=216820&view=diff</a><br>
> ==============================================================================<br>
> --- llvm/trunk/lib/Transforms/Instrumentation/DataFlowSanitizer.cpp (original)<br>
> +++ llvm/trunk/lib/Transforms/Instrumentation/DataFlowSanitizer.cpp Sat Aug 30 11:48:02 2014<br>
> @@ -143,7 +143,7 @@ class DFSanABIList {<br>
><br>
>Â Â /// Returns whether either this function or its source file are listed in the<br>
>Â Â /// given category.<br>
> -Â bool isIn(const Function &F, const StringRef Category) const {<br>
> +Â bool isIn(const Function &F, StringRef Category) const {<br>
>Â Â Â return isIn(*F.getParent(), Category) ||<br>
>Â Â Â Â Â Â Â SCL->inSection("fun", F.getName(), Category);<br>
>Â Â }<br>
> @@ -152,7 +152,7 @@ class DFSanABIList {<br>
>Â Â ///<br>
>Â Â /// If GA aliases a function, the alias's name is matched as a function name<br>
>  /// would be. Similarly, aliases of globals are matched like globals.<br>
> -Â bool isIn(const GlobalAlias &GA, const StringRef Category) const {<br>
> +Â bool isIn(const GlobalAlias &GA, StringRef Category) const {<br>
>Â Â Â if (isIn(*GA.getParent(), Category))<br>
>Â Â Â Â return true;<br>
><br>
> @@ -164,7 +164,7 @@ class DFSanABIList {<br>
>Â Â }<br>
><br>
>Â Â /// Returns whether this module is listed in the given category.<br>
> -Â bool isIn(const Module &M, const StringRef Category) const {<br>
> +Â bool isIn(const Module &M, StringRef Category) const {<br>
>Â Â Â return SCL->inSection("src", M.getModuleIdentifier(), Category);<br>
>Â Â }<br>
>Â };<br>
><br>
> Modified: llvm/trunk/tools/llvm-dwarfdump/llvm-dwarfdump.cpp<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/llvm-dwarfdump/llvm-dwarfdump.cpp?rev=216820&r1=216819&r2=216820&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/llvm-dwarfdump/llvm-dwarfdump.cpp?rev=216820&r1=216819&r2=216820&view=diff</a><br>
> ==============================================================================<br>
> --- llvm/trunk/tools/llvm-dwarfdump/llvm-dwarfdump.cpp (original)<br>
> +++ llvm/trunk/tools/llvm-dwarfdump/llvm-dwarfdump.cpp Sat Aug 30 11:48:02 2014<br>
> @@ -65,7 +65,7 @@ DumpType("debug-dump", cl::init(DIDT_All<br>
>Â Â Â Â Â clEnumValN(DIDT_StrOffsetsDwo, "str_offsets.dwo", ".debug_str_offsets.dwo"),<br>
>Â Â Â Â Â clEnumValEnd));<br>
><br>
> -static void DumpInput(const StringRef &Filename) {<br>
> +static void DumpInput(StringRef Filename) {<br>
>Â Â ErrorOr<std::unique_ptr<MemoryBuffer>> BuffOrErr =<br>
>Â Â Â Â MemoryBuffer::getFileOrSTDIN(Filename);<br>
><br>
><br>
><br>
> _______________________________________________<br>
> llvm-commits mailing list<br>
> <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br>~Craig
</div>