mhjacobson updated this revision to Diff 476017.
mhjacobson added a comment.
Run clang-format.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D138183/new/
https://reviews.llvm.org/D138183
Files:
clang/lib/Driver/ToolChains/FreeBSD.cpp
clang/lib/
mhjacobson updated this revision to Diff 476018.
mhjacobson added a comment.
Clarify comment.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D138183/new/
https://reviews.llvm.org/D138183
Files:
clang/lib/Driver/ToolChains/FreeBSD.cpp
clang/lib/D
owenpan added a comment.
@gedare thanks for changing `EXPECT_EQ` to `verifyFormat`, but IMO we should do
that in another patch so that it would be easier (at least for me) to review
the new tests and to make sure no existing tests have been changed by accident.
Comment at: cl
MaskRay added inline comments.
Comment at: clang/lib/Driver/ToolChains/FreeBSD.cpp:442
+
+ addExternCSystemInclude(DriverArgs, CC1Args,
+ concat(D.SysRoot, "/usr/include"));
I think Fuchsia way of checking `if (!D.SysRoot.empty()) {` bef
MaskRay added a comment.
test?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D138179/new/
https://reviews.llvm.org/D138179
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cg
mhjacobson added inline comments.
Comment at: clang/lib/Driver/ToolChains/FreeBSD.cpp:442
+
+ addExternCSystemInclude(DriverArgs, CC1Args,
+ concat(D.SysRoot, "/usr/include"));
MaskRay wrote:
> I think Fuchsia way of checking `if (!D.Sys
201 - 206 of 206 matches
Mail list logo