[PATCH] D92176: Don't use sysroot/include when sysroot is empty.

This revision was automatically updated to reflect the committed changes. Closed by commit rGca2888310b24: Dont use sysroot/include when sysroot is empty. (authored by abidh). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D92176/new/

[PATCH] D92176: Don't use sysroot/include when sysroot is empty.

jroelofs accepted this revision. jroelofs added a comment. This revision is now accepted and ready to land. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D92176/new/ https://reviews.llvm.org/D92176

[PATCH] D92176: Don't use sysroot/include when sysroot is empty.

abidh added a comment. @jroelofs Do you have any comments on this patch? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D92176/new/ https://reviews.llvm.org/D92176 ___ cfe-commits mailing list

[PATCH] D92176: Don't use sysroot/include when sysroot is empty.

cjdb added a comment. In D92176#2419011 , @manojgupta wrote: > lgtm but not an expert in this area. Ditto Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D92176/new/ https://reviews.llvm.org/D92176

[PATCH] D92176: Don't use sysroot/include when sysroot is empty.

manojgupta added a comment. lgtm but not an expert in this area. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D92176/new/ https://reviews.llvm.org/D92176 ___ cfe-commits mailing list

[PATCH] D92176: Don't use sysroot/include when sysroot is empty.

abidh created this revision. abidh added reviewers: jroelofs, manojgupta, cjdb. abidh added a project: clang. Herald added a subscriber: cfe-commits. abidh requested review of this revision. Baremetal toolchain add Driver.SysRoot/include to the system include paths without checking if