[PATCH] D21810: Don't instantiate a full host toolchain in ASTMatchersTest.

Chandler Carruth via cfe-commits cfe-commits at lists.llvm.org
Thu Jun 30 11:17:36 PDT 2016


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

In http://reviews.llvm.org/D21810#471486, @jlebar wrote:

> > But I think this is a reasonable workaround until such an API can be provided.
>
>
> Should I take that as an LG, or are we waiting for someone else to approve this?


Lacking a more detailed response from Manuel, yea, I think you should check this in... I wrote most of the toolchain search logic you're turning off so I'm happy to provide an LGTM here. I think this is fine for now, fixes an immediate issue, and can be revisited and improved later easily. So yes, LG.


http://reviews.llvm.org/D21810





More information about the cfe-commits mailing list