sammccall accepted this revision. sammccall added inline comments. This revision is now accepted and ready to land.
================ Comment at: include/llvm/Support/YAMLTraits.h:524 + // ambiguity with other YAML constructs. + // FIXME: However, the “:”, “?” and “-” indicators may be used as the first + // character if followed by a non-space “safe” character, as this causes no ---------------- Up to you, but I'd leave this comment out, or rephrase as a non-fixme - there's nothing *wrong* with the new behavior. Repository: rL LLVM https://reviews.llvm.org/D42362 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
