<div dir="auto"><div><div class="gmail_extra"><div class="gmail_quote">On 19 Nov 2016 2:36 am, "Anton Bikineev" <<a href="mailto:ant.bikineev@gmail.com">ant.bikineev@gmail.com</a>> wrote:<br type="attribution"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">AntonBikineev added inline comments.<br>
<br>
<br>
================<br>
<div class="quoted-text">Comment at: lib/Lex/LiteralSupport.cpp:<wbr>1716-1717<br>
+StringLiteralParser::<wbr>UDSuffixResult<br>
+StringLiteralParser::<wbr>isValidUDSuffix(const LangOptions &LangOpts,<br>
+                                     StringRef Suffix) {<br>
+  if (!LangOpts.CPlusPlus11 || Suffix.empty())<br>
----------------<br>
</div><div class="quoted-text">rsmith wrote:<br>
> Just make this call `NumericLiteralParser::<wbr>isValidUDSuffix` and then check for the `sv` case. All the numeric suffixes are also valid string literal suffixes for the form `operator""suffix`.<br>
</div>This makes sense for the call sites we currently have.<br>
> All the numeric suffixes are also valid string literal suffixes for the form operator""suffix.<br>
Don't really understand this part. It seems inconsistent if one calls, say,<br>
``` StringLiteralParser::<wbr>isValidUDSuffix(LangOpts(), "if")  ```<br>
 and gets<br>
```true```<br></blockquote></div></div></div><div dir="auto"><br></div><div dir="auto"><span style="font-family:sans-serif">That is valid in the case of `operator ""if`, so `if` is a valid string literal suffix (as is any numeric literal suffix).</span><br></div><div dir="auto"><br></div><div dir="auto"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<a href="https://reviews.llvm.org/D26829" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D26829</a><br>
<br>
<br>
<br>
</blockquote></div><br></div></div></div>