[PATCH] D25153: preprocessor supports `-dI` flag

Steve O'Brien via cfe-commits cfe-commits at lists.llvm.org
Mon Oct 3 17:48:42 PDT 2016


elsteveogrande added inline comments.


> elsteveogrande wrote in PrintPreprocessedOutput.cpp:320-321
> right you are!  I was thinking of `//` when I was doing that escaping logic.
> 
> I should either escape `*/`, or even better, just use `//` for comments, is that ok?  (I feel like `//` is more robust because it'll really ignore characters until the end-of-line.)

never mind, can't use // with c89, ansi, etc.  Will correct this escaping business.

Repository:
  rL LLVM

https://reviews.llvm.org/D25153





More information about the cfe-commits mailing list