hamzasood updated this revision to Diff 86900.
hamzasood added a comment.
In https://reviews.llvm.org/D28365#665183, @rnk wrote:
> I had to revert this because it doesn't pass tests on Linux. Can you look
> into that and resubmit after fixing those test failures?
Really sorry about that, I stu
hamzasood created this revision.
While using the libtooling API, I ran into problems when using a const ASTUnit.
There were a few functions I needed to use that don't seem to modify anything,
but they weren't marked as const so I couldn't use them.
This patch marks those methods (and a few othe
hamzasood added a comment.
Could someone commit this for me? I don't have access.
https://reviews.llvm.org/D35729
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
hamzasood added a comment.
In https://reviews.llvm.org/D28365#665408, @rnk wrote:
> Doesn't your fix mean that the tests will fail on a Windows machine that
> doesn't have VS because LLVM was built with mingw? Usually in these
> situations we provide some way to provide a fake toolchain.
I'm
101 - 104 of 104 matches
Mail list logo