dberris accepted this revision.
dberris added a comment.
This revision is now accepted and ready to land.

In https://reviews.llvm.org/D45716#1069567, @dberris wrote:

> In https://reviews.llvm.org/D45716#1069557, @kpw wrote:
>
> > My editor got a bit carried away with automatically clang-formatting 
> > lib/CodeGen/CodeGenFunction.cpp. I'll fix that so that I'm not messing up 
> > the revision history.
>
>
> Yes please. :)
>
> Also, if you can add to the tests for instrumentation bundles in 
> `test/Driver/XRay/...` to ensure that we're properly not seeing the typed 
> events there either, that would be great.


Actually, I see that change too. Ignore that. :)

LGTM (pending the revert of formatting changes in 
`lib/CodeGen/CodeGenFunction.cpp`).


Repository:
  rC Clang

https://reviews.llvm.org/D45716



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

Reply via email to