Going to have to block this.  I tried to add a UDL recently and was blocked on the grounds that UDLs are not welcome in the codebase, so i guess the same applies here<br><div class="gmail_quote"><div dir="ltr">On Sun, Nov 6, 2016 at 4:46 AM Paweł Żukowski <<a href="mailto:p.z.idlecode@gmail.com">p.z.idlecode@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">idlecode added inline comments.<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
================<br class="gmail_msg">
Comment at: include/llvm/ADT/StringRef.h:778<br class="gmail_msg">
+<br class="gmail_msg">
+  namespace operators {<br class="gmail_msg">
+  /// User defined literal<br class="gmail_msg">
----------------<br class="gmail_msg">
Does this operator need it's own namespace?<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
================<br class="gmail_msg">
Comment at: lib/CodeGen/AsmPrinter/AsmPrinter.cpp:1221<br class="gmail_msg">
     MCSymbol *AddrSymbol =<br class="gmail_msg">
-        OutContext.getOrCreateSymbol(StringRef("__morestack_addr"));<br class="gmail_msg">
+        OutContext.getOrCreateSymbol("__morestack_addr"_sr);<br class="gmail_msg">
     OutStreamer->EmitLabel(AddrSymbol);<br class="gmail_msg">
----------------<br class="gmail_msg">
`getOrCreateSymbol` takes Twine as argument so `StringRef` conversion is not needed.<br class="gmail_msg">
There is few other such use-cases but it would be quite tedious to fix them all manually - maybe clang-tidy could take care of it in the future.<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<a href="https://reviews.llvm.org/D26332" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D26332</a><br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
</blockquote></div>