mclow.lists added inline comments.

================
Comment at: lib/Lex/LiteralSupport.cpp:768
       .Cases("il", "i", "if", true)
+      .Case("sv", true)
       .Default(false);
----------------
malcolm.parsons wrote:
> This is in `NumericLiteralParser::isValidUDSuffix()`.
> 
> If a change is needed for `"sv"`, wouldn't it be in `StringLiteralParser`?
Would it be?  I don't know.  `"s"` appears here, which is used for both 
'string' and 'seconds'.


https://reviews.llvm.org/D26667



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to