[PATCH] [extra] pptrace - preprocessor tracing and testing tool

John Thompson john.thompson.jtsoftware at gmail.com
Mon Oct 28 17:52:04 PDT 2013


  Sean,

  Thanks for the excellent review, and the reference to yaml-bench.  It caught a few things so far, and I'll have to use it when I write the other tests too.  It didn't like the backslashes in Windows file paths, or the colons in SourceLocations, so I put in some character replacing and quoting.

  I revised the for loops as pointed out.

  Also, I wanted to ask about the special cases, such as where I use values like (null) to represent when a pointer is null, or (invalid) when a SourceLocation has the invalid flag set.  In the former case, passing a null pointer is expected in some situations.  The YAML parser doesn't seem to mind (null), but is this reasonable semantically?

  Also, again semantically, you probably already noticed I'm not trying to flatten the full data structures for possible later resurrection, just providing what I think is sufficient high-level information to know what the preprocessor is up to.

  Kim,

  Thanks also for your help. I'll install ninja cmake and give it a shot.

  -John

Hi silvas, kimgr,

http://llvm-reviews.chandlerc.com/D2020

CHANGE SINCE LAST DIFF
  http://llvm-reviews.chandlerc.com/D2020?vs=5196&id=5210#toc

Files:
  test/pp-trace/pp-trace-macro.cpp
  pp-trace/PPTrace.cpp
  pp-trace/PPCallbacksTracker.h
  pp-trace/PPCallbacksTracker.cpp
  pp-trace/CMakeLists.txt
  pp-trace/Makefile
  CMakeLists.txt
  Makefile
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D2020.4.patch
Type: text/x-patch
Size: 48506 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20131028/36cd1098/attachment.bin>


More information about the cfe-commits mailing list