jeremyd2019 wrote:

> > I rebased this on top of #138783 and adjusted the title and description. 
> > Now it should be in a good state to push cmake changes for other projects.
> 
> The changes look good, but it looks like the changes from #138783 still show 
> up when viewing the changes; can you check that you've rebased past the 
> merged #138783?

I had not - I have now though.

> (Also, I take it that no other subprojects than clang need the 
> `cmake_push_check_state` change?)

No, the other projects were not messing with `_GNU_SOURCE` like clang was.

https://github.com/llvm/llvm-project/pull/138587
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to