[PATCH] D26654: [CMake] Add Fuchsia toolchain CMake cache files

phosek updated this revision to Diff 87339. Repository: rL LLVM https://reviews.llvm.org/D26654 Files: cmake/caches/Fuchsia-stage2.cmake cmake/caches/Fuchsia.cmake Index: cmake/caches/Fuchsia.cmake === --- /dev/null +++

[PATCH] D28974: [Basic] Remove the 32-bit x86 and ARM targets for Fuchsia

phosek created this revision. Herald added subscribers: rengolin, aemerson. The 32-bit architectures are no longer supported by Fuchsia. Repository: rL LLVM https://reviews.llvm.org/D28974 Files: lib/Basic/Targets.cpp Index: lib/Basic/Targets.cpp

[PATCH] D28974: [Basic] Remove the 32-bit x86 and ARM targets for Fuchsia

This revision was automatically updated to reflect the committed changes. Closed by commit rL292671: [Basic] Remove the 32-bit x86 and ARM targets for Fuchsia (authored by phosek). Changed prior to commit: https://reviews.llvm.org/D28974?vs=85199=85205#toc Repository: rL LLVM

[PATCH] D30238: [Driver] Enable SafeStack for Fuchsia targets

phosek added a comment. Could you also please update test/Driver/fuchsia.c and add a case for `-fsanitize=safe-stack`? Repository: rL LLVM https://reviews.llvm.org/D30238 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D26654: [CMake] Add Fuchsia toolchain CMake cache files

This revision was automatically updated to reflect the committed changes. Closed by commit rL295480: [CMake] Add Fuchsia toolchain CMake cache files (authored by phosek). Changed prior to commit: https://reviews.llvm.org/D26654?vs=88770=88958#toc Repository: rL LLVM

[PATCH] D30238: [Driver] Enable SafeStack for Fuchsia targets

phosek accepted this revision. phosek added a comment. This revision is now accepted and ready to land. LGTM Repository: rL LLVM https://reviews.llvm.org/D30238 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D30238: [Driver] Enable SafeStack for Fuchsia targets

This revision was automatically updated to reflect the committed changes. Closed by commit rL296082: [Driver] Enable SafeStack for Fuchsia targets (authored by phosek). Changed prior to commit: https://reviews.llvm.org/D30238?vs=89393=89607#toc Repository: rL LLVM

[PATCH] D29843: [CodeGen] Treat auto-generated __dso_handle symbol as HiddenVisibility

phosek accepted this revision. phosek added a comment. This revision is now accepted and ready to land. LGTM https://reviews.llvm.org/D29843 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D29843: [CodeGen] Treat auto-generated __dso_handle symbol as HiddenVisibility

phosek added a comment. In https://reviews.llvm.org/D29843#674111, @mcgrathr wrote: > This is for clang, not llvm. Should it be cfe-commits instead of > llvm-commits? Right, my mistake. https://reviews.llvm.org/D29843 ___ cfe-commits mailing

[PATCH] D29843: [CodeGen] Treat auto-generated __dso_handle symbol as HiddenVisibility

phosek added a comment. You can use `ninja check-clang` to run Clang tests, if those pass you can use `ninja check-all` to run all tests. Testing guide has details about testing in LLVM. https://reviews.llvm.org/D29843

[PATCH] D29843: [CodeGen] Treat auto-generated __dso_handle symbol as HiddenVisibility

phosek added a comment. Did you run all tests? I'd suspect that this might break `test/OpenMP/threadprivate_codegen.cpp` which is expecting `@__dso_handle = external global i8` which will now be `hidden global`? https://reviews.llvm.org/D29843

[PATCH] D26654: [CMake] Add Fuchsia toolchain CMake cache files

phosek updated this revision to Diff 88770. Repository: rL LLVM https://reviews.llvm.org/D26654 Files: cmake/caches/Fuchsia-stage2.cmake cmake/caches/Fuchsia.cmake Index: cmake/caches/Fuchsia.cmake === --- /dev/null +++

[PATCH] D26654: [CMake] Add Fuchsia toolchain CMake cache files

phosek updated this revision to Diff 88575. Repository: rL LLVM https://reviews.llvm.org/D26654 Files: cmake/caches/Fuchsia-stage2.cmake cmake/caches/Fuchsia.cmake Index: cmake/caches/Fuchsia.cmake === --- /dev/null +++

[PATCH] D28821: [CMake] Separate LLVM_ENABLE_LTO and LLVM_ENABLE_LLD

This revision was automatically updated to reflect the committed changes. Closed by commit rL292343: [CMake] Separate LLVM_ENABLE_LTO and LLVM_ENABLE_LLD (authored by phosek). Changed prior to commit: https://reviews.llvm.org/D28821?vs=84716=84798#toc Repository: rL LLVM

[PATCH] D28821: [CMake] Separate LLVM_ENABLE_LTO and LLVM_ENABLE_LLD

phosek created this revision. Herald added a subscriber: mgorny. These two are options independent: it's possible to use LLD without LTO, and it's possible to do LTO build without LLD. Repository: rL LLVM https://reviews.llvm.org/D28821 Files: CMakeLists.txt Index: CMakeLists.txt

[PATCH] D26649: [CMake] Support lld with LTO bootstrap

phosek added inline comments. Comment at: cfe/trunk/CMakeLists.txt:531 + if(BOOTSTRAP_LLVM_ENABLE_LLD) +add_dependencies(clang-bootstrap-deps lld) + elseif(LLVM_BINUTILS_INCDIR) beanz wrote: > mehdi_amini wrote: > > I come back to this a bit

[PATCH] D25263: [Driver] Allow setting the default linker during build

phosek updated this revision to Diff 81276. phosek marked 3 inline comments as done. Repository: rL LLVM https://reviews.llvm.org/D25263 Files: CMakeLists.txt include/clang/Config/config.h.cmake include/clang/Driver/ToolChain.h lib/Driver/ToolChain.cpp lib/Driver/ToolChains.cpp

[PATCH] D25263: [Driver] Allow setting the default linker during build

phosek updated this revision to Diff 80934. phosek marked 4 inline comments as done. Repository: rL LLVM https://reviews.llvm.org/D25263 Files: CMakeLists.txt include/clang/Config/config.h.cmake include/clang/Driver/ToolChain.h lib/Driver/ToolChain.cpp lib/Driver/ToolChains.cpp

[PATCH] D25263: [Driver] Allow setting the default linker during build

phosek added inline comments. Comment at: lib/Driver/ToolChain.cpp:362 + return UseLinker; + } else if (UseLinker == "ld") { +// If we're passed the argument ld, then use whatever the default system I'm wandering whether we shouldn't use `"platform"`

[PATCH] D32231: [CMake] Enable ARM target in Fuchsia toolchain

This revision was automatically updated to reflect the committed changes. Closed by commit rL300917: [CMake] Enable ARM target in Fuchsia toolchain (authored by phosek). Changed prior to commit: https://reviews.llvm.org/D32231?vs=95766=96048#toc Repository: rL LLVM

[PATCH] D32231: [CMake] Enable ARM target in Fuchsia toolchain

phosek updated this revision to Diff 95766. Repository: rL LLVM https://reviews.llvm.org/D32231 Files: cmake/caches/Fuchsia-stage2.cmake Index: cmake/caches/Fuchsia-stage2.cmake === --- cmake/caches/Fuchsia-stage2.cmake +++

[PATCH] D32231: [CMake] Enable ARM target in Fuchsia toolchain

phosek created this revision. Herald added subscribers: mgorny, rengolin, aemerson. This is still used by some users of Fuchsia toolchain. Also include llc and opt which is useful for development and testing. Repository: rL LLVM https://reviews.llvm.org/D32231 Files:

[PATCH] D36194: [CMake] Include llvm-objcopy tool in Fuchsia toolchain

This revision was automatically updated to reflect the committed changes. Closed by commit rL309786: [CMake] Include llvm-objcopy tool in Fuchsia toolchain (authored by phosek). Changed prior to commit: https://reviews.llvm.org/D36194?vs=109233=109265#toc Repository: rL LLVM

[PATCH] D35922: [Driver] Enable AddressSanitizer for Fuchsia targets

phosek added a comment. One more thing, would it be possible add a test case to `test/Driver/fuchsia.c` covering this? Repository: rL LLVM https://reviews.llvm.org/D35922 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D35922: [Driver] Enable AddressSanitizer for Fuchsia targets

phosek added inline comments. Comment at: cmake/caches/Fuchsia-stage2.cmake:50 set(RUNTIMES_${target}-fuchsia_LIBCXX_ABI_VERSION 2 CACHE STRING "") + set(RUNTIMES_${target}-fuchsia_SANITIZER_USE_COMPILER_RT ON CACHE BOOL "") endforeach() mcgrathr wrote: >

[PATCH] D35922: [Driver] Enable AddressSanitizer for Fuchsia targets

phosek accepted this revision. phosek added a comment. This revision is now accepted and ready to land. LGTM Repository: rL LLVM https://reviews.llvm.org/D35922 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D36202: [Driver] Disable static C++ library support on Fuchsia

This revision was automatically updated to reflect the committed changes. Closed by commit rL309778: [Driver] Disable static C++ library support on Fuchsia (authored by phosek). Changed prior to commit: https://reviews.llvm.org/D36202?vs=109255=109258#toc Repository: rL LLVM

[PATCH] D36202: [Driver] Disable static C++ library support on Fuchsia

phosek added inline comments. Comment at: lib/Driver/ToolChains/Fuchsia.cpp:112 ToolChain.AddCXXStdlibLibArgs(Args, CmdArgs); -if (OnlyLibstdcxxStatic) - CmdArgs.push_back("-Bdynamic"); - } CmdArgs.push_back("-lm"); }

[PATCH] D36202: [Driver] Disable static C++ library support on Fuchsia

phosek created this revision. Herald added a subscriber: mgorny. Don't support or build static C++ libraries for Fuchsia. Repository: rL LLVM https://reviews.llvm.org/D36202 Files: cmake/caches/Fuchsia-stage2.cmake lib/Driver/ToolChains/Fuchsia.cpp test/Driver/fuchsia.cpp Index:

[PATCH] D35922: [Driver] Enable AddressSanitizer for Fuchsia targets

phosek accepted this revision. phosek added a comment. LGTM https://reviews.llvm.org/D35922 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D35820: [Driver] Search compiler-rt paths in -print-file-name=

phosek updated this revision to Diff 109485. Repository: rL LLVM https://reviews.llvm.org/D35820 Files: include/clang/Driver/ToolChain.h lib/Driver/Driver.cpp lib/Driver/ToolChain.cpp Index: lib/Driver/ToolChain.cpp ===

[PATCH] D36194: [CMake] Include llvm-objcopy tool in Fuchsia toolchain

phosek created this revision. Herald added a subscriber: mgorny. Repository: rL LLVM https://reviews.llvm.org/D36194 Files: cmake/caches/Fuchsia-stage2.cmake Index: cmake/caches/Fuchsia-stage2.cmake === ---

[PATCH] D36254: [Driver][Fuchsia] Pass --hash-style=gnu to the linker

This revision was automatically updated to reflect the committed changes. Closed by commit rL310017: [Driver][Fuchsia] Pass --hash-style=gnu to the linker (authored by phosek). Changed prior to commit: https://reviews.llvm.org/D36254?vs=109603=109677#toc Repository: rL LLVM

[PATCH] D35922: [Driver] Enable AddressSanitizer for Fuchsia targets

This revision was automatically updated to reflect the committed changes. Closed by commit rL30: Enable AddressSanitizer for Fuchsia targets (authored by phosek). Changed prior to commit: https://reviews.llvm.org/D35922?vs=109605=109645#toc Repository: rL LLVM

[PATCH] D36254: [Driver][Fuchsia] Pass --hash-style=gnu to the linker

phosek accepted this revision. phosek added a comment. This revision is now accepted and ready to land. LGTM https://reviews.llvm.org/D36254 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D36541: [CMake] Include LLVMFuzzer in Fuchsia toolchain

This revision was automatically updated to reflect the committed changes. Closed by commit rL310901: [CMake] Include LLVMFuzzer in Fuchsia toolchain (authored by phosek). Changed prior to commit: https://reviews.llvm.org/D36541?vs=110452=03#toc Repository: rL LLVM

[PATCH] D35820: [Driver] Search compiler-rt paths in -print-file-name=

This revision was automatically updated to reflect the committed changes. Closed by commit rL310548: [Driver] Search compiler-rt paths in -print-file-name= (authored by phosek). Changed prior to commit: https://reviews.llvm.org/D35820?vs=109485=110515#toc Repository: rL LLVM

[PATCH] D32817: [CMake] Build runtimes for Fuchsia targets

phosek updated this revision to Diff 106098. Repository: rL LLVM https://reviews.llvm.org/D32817 Files: cmake/caches/Fuchsia-stage2.cmake cmake/caches/Fuchsia.cmake Index: cmake/caches/Fuchsia.cmake === ---

[PATCH] D32817: [CMake] Build runtimes for Fuchsia targets

phosek updated this revision to Diff 106349. Repository: rL LLVM https://reviews.llvm.org/D32817 Files: cmake/caches/Fuchsia-stage2.cmake cmake/caches/Fuchsia.cmake Index: cmake/caches/Fuchsia.cmake === ---

[PATCH] D35603: [Driver] Consider -fno-sanitize=... state when filtering out -fsanitize-coverage=...

phosek accepted this revision. phosek added a comment. This revision is now accepted and ready to land. LGTM Repository: rL LLVM https://reviews.llvm.org/D35603 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D35582: [Driver] Always use -z rodynamic for Fuchsia

phosek created this revision. Fuchsia uses read-only .dynamic section. Repository: rL LLVM https://reviews.llvm.org/D35582 Files: lib/Driver/ToolChains/Fuchsia.cpp test/Driver/fuchsia.c test/Driver/fuchsia.cpp Index: test/Driver/fuchsia.cpp

[PATCH] D35514: [CMake] Use MATCHES for regular expression comparison.

phosek accepted this revision. phosek added a comment. This revision is now accepted and ready to land. LGTM Repository: rL LLVM https://reviews.llvm.org/D35514 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D32817: [CMake] Build runtimes for Fuchsia targets

phosek updated this revision to Diff 107178. Repository: rL LLVM https://reviews.llvm.org/D32817 Files: cmake/caches/Fuchsia-stage2.cmake cmake/caches/Fuchsia.cmake Index: cmake/caches/Fuchsia.cmake === ---

[PATCH] D35582: [Driver] Always use -z rodynamic for Fuchsia

phosek updated this revision to Diff 107185. phosek marked an inline comment as done. Repository: rL LLVM https://reviews.llvm.org/D35582 Files: lib/Driver/ToolChains/Fuchsia.cpp test/Driver/fuchsia.c test/Driver/fuchsia.cpp Index: test/Driver/fuchsia.cpp

[PATCH] D35582: [Driver] Always use -z rodynamic for Fuchsia

This revision was automatically updated to reflect the committed changes. Closed by commit rL308390: [Driver] Always use -z rodynamic for Fuchsia (authored by phosek). Changed prior to commit: https://reviews.llvm.org/D35582?vs=107185=107200#toc Repository: rL LLVM

[PATCH] D34489: [scan-build-py] Patch to fix "-analyzer-config" option

This revision was automatically updated to reflect the committed changes. Closed by commit rL308401: [scan-build-py] Patch to fix "-analyzer-config" option (authored by phosek). Changed prior to commit: https://reviews.llvm.org/D34489?vs=103490=107214#toc Repository: rL LLVM

[PATCH] D32817: [CMake] Build runtimes for Fuchsia targets

This revision was automatically updated to reflect the committed changes. Closed by commit rL308412: [CMake] Build runtimes for Fuchsia targets (authored by phosek). Changed prior to commit: https://reviews.llvm.org/D32817?vs=107236=107240#toc Repository: rL LLVM

[PATCH] D32817: [CMake] Build runtimes for Fuchsia targets

phosek updated this revision to Diff 107236. Repository: rL LLVM https://reviews.llvm.org/D32817 Files: cmake/caches/Fuchsia-stage2.cmake cmake/caches/Fuchsia.cmake Index: cmake/caches/Fuchsia.cmake === ---

[PATCH] D35705: [CMake] Use ABI version 2 for C++ library

phosek created this revision. Herald added a subscriber: mgorny. Repository: rL LLVM https://reviews.llvm.org/D35705 Files: cmake/caches/Fuchsia-stage2.cmake Index: cmake/caches/Fuchsia-stage2.cmake === ---

[PATCH] D35603: [Driver] Consider -fno-sanitize=... state when filtering out -fsanitize-coverage=...

This revision was automatically updated to reflect the committed changes. Closed by commit rL308707: [Driver] Consider -fno-sanitize=... state when filtering out -fsanitize… (authored by phosek). Changed prior to commit: https://reviews.llvm.org/D35603?vs=107234=107620#toc Repository: rL

[PATCH] D35705: [CMake] Use ABI version 2 for C++ library in Fuchsia

This revision was automatically updated to reflect the committed changes. Closed by commit rL308705: [CMake] Use ABI version 2 for C++ library in Fuchsia (authored by phosek). Changed prior to commit: https://reviews.llvm.org/D35705?vs=107600=107617#toc Repository: rL LLVM

[PATCH] D35747: [Driver] Fuchsia defaults to -fno-math-errno

phosek accepted this revision. phosek added a comment. This revision is now accepted and ready to land. LGTM https://reviews.llvm.org/D35747 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D35820: [Driver] Search compiler-rt paths in -print-file-name=

phosek created this revision. Herald added a subscriber: dberris. This makes it possible to print the name of compiler-rt libraries by using simply clang -print-file-name=libclang_rt.${runtime}-${arch}.so same as other libraries, without having to know the details of the resource directory

[PATCH] D35747: [Driver] Fuchsia defaults to -fno-math-errno

This revision was automatically updated to reflect the committed changes. Closed by commit rL308855: [Driver] Fuchsia defaults to -fno-math-errno (authored by phosek). Changed prior to commit: https://reviews.llvm.org/D35747?vs=107752=107845#toc Repository: rL LLVM

[PATCH] D35930: [CMake] Include sancov tool in Fuchsia toolchain

phosek created this revision. Herald added a subscriber: mgorny. Repository: rL LLVM https://reviews.llvm.org/D35930 Files: cmake/caches/Fuchsia-stage2.cmake Index: cmake/caches/Fuchsia-stage2.cmake === ---

[PATCH] D35922: [Driver] Enable AddressSanitizer for Fuchsia targets

phosek added inline comments. Comment at: cmake/caches/Fuchsia-stage2.cmake:50 set(RUNTIMES_${target}-fuchsia_LIBCXX_ABI_VERSION 2 CACHE STRING "") + set(RUNTIMES_${target}-fuchsia_SANITIZER_USE_COMPILER_RT ON CACHE BOOL "") endforeach() I think we also

[PATCH] D35328: Reland "[Driver] Update Fuchsia driver path handling"

phosek updated this revision to Diff 106323. phosek marked an inline comment as done. Repository: rL LLVM https://reviews.llvm.org/D35328 Files: lib/Driver/ToolChains/Fuchsia.cpp lib/Driver/ToolChains/Fuchsia.h test/Driver/fuchsia.c test/Driver/fuchsia.cpp Index:

[PATCH] D35328: Reland "[Driver] Update Fuchsia driver path handling"

This revision was automatically updated to reflect the committed changes. Closed by commit rL307856: Reland "[Driver] Update Fuchsia driver path handling" (authored by phosek). Changed prior to commit: https://reviews.llvm.org/D35328?vs=106323=106327#toc Repository: rL LLVM

[PATCH] D35328: Reland "[Driver] Update Fuchsia driver path handling"

phosek created this revision. Several improvements to the Fuchsia driver: - Search for C++ library headers and libraries in directories that are part of the toolchain distribution rather than sysroot. - Use LLVM support utlities to construct paths to make sure the driver is also usable on

[PATCH] D32817: [CMake] Build runtimes for Fuchsia targets

phosek updated this revision to Diff 106311. Repository: rL LLVM https://reviews.llvm.org/D32817 Files: cmake/caches/Fuchsia-stage2.cmake cmake/caches/Fuchsia.cmake Index: cmake/caches/Fuchsia.cmake === ---

[PATCH] D35328: Reland "[Driver] Update Fuchsia driver path handling"

phosek added a comment. I have fixed the test to also handle Windows paths properly which was causing a build breakage. Repository: rL LLVM https://reviews.llvm.org/D35328 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D32613: [Driver] Update Fuchsia driver path handling

phosek updated this revision to Diff 106275. Repository: rL LLVM https://reviews.llvm.org/D32613 Files: lib/Driver/ToolChains/Fuchsia.cpp lib/Driver/ToolChains/Fuchsia.h test/Driver/fuchsia.cpp Index: test/Driver/fuchsia.cpp

[PATCH] D32613: [Driver] Update Fuchsia driver path handling

This revision was automatically updated to reflect the committed changes. Closed by commit rL307830: [Driver] Update Fuchsia driver path handling (authored by phosek). Changed prior to commit: https://reviews.llvm.org/D32613?vs=106275=106276#toc Repository: rL LLVM

[PATCH] D34489: [scan-build-py] Patch to fix "-analyzer-config" option

phosek accepted this revision. phosek added a comment. This revision is now accepted and ready to land. LGTM https://reviews.llvm.org/D34489 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D32613: [Driver] Update Fuchsia driver path handling

phosek created this revision. Several improvements to the Fuchsia driver: - Search for C++ library headers and libraries in directories that are part of the toolchain distribution rather than sysroot. - Use LLVM support utlities to construct paths to make sure the driver is also usable

[PATCH] D36349: [CMake] Build sanitized C++ runtimes for Fuchsia

phosek created this revision. Herald added a subscriber: mgorny. Repository: rL LLVM https://reviews.llvm.org/D36349 Files: cmake/caches/Fuchsia-stage2.cmake Index: cmake/caches/Fuchsia-stage2.cmake === ---

[PATCH] D36349: [CMake] Build sanitized C++ runtimes for Fuchsia

phosek updated this revision to Diff 109852. Repository: rL LLVM https://reviews.llvm.org/D36349 Files: cmake/caches/Fuchsia-stage2.cmake Index: cmake/caches/Fuchsia-stage2.cmake === --- cmake/caches/Fuchsia-stage2.cmake +++

[PATCH] D36349: [CMake] Build sanitized C++ runtimes for Fuchsia

phosek updated this revision to Diff 110311. Repository: rL LLVM https://reviews.llvm.org/D36349 Files: cmake/caches/Fuchsia-stage2.cmake Index: cmake/caches/Fuchsia-stage2.cmake === --- cmake/caches/Fuchsia-stage2.cmake +++

[PATCH] D35930: [CMake] Include sancov tool in Fuchsia toolchain

This revision was automatically updated to reflect the committed changes. Closed by commit rL309724: [CMake] Include sancov tool in Fuchsia toolchain (authored by phosek). Changed prior to commit: https://reviews.llvm.org/D35930?vs=108418=109173#toc Repository: rL LLVM

[PATCH] D36349: [CMake] Build sanitized C++ runtimes for Fuchsia

phosek added inline comments. Comment at: cmake/caches/Fuchsia-stage2.cmake:56 +foreach(target x86_64;aarch64) + set(RUNTIMES_${target}-fuchsia-asan_CMAKE_BUILD_WITH_INSTALL_RPATH ON CACHE BOOL "") mcgrathr wrote: > Can you do this without duplicating all

[PATCH] D36349: [CMake] Build sanitized C++ runtimes for Fuchsia

This revision was automatically updated to reflect the committed changes. Closed by commit rL311033: [CMake] Build sanitized C++ runtimes for Fuchsia (authored by phosek). Changed prior to commit: https://reviews.llvm.org/D36349?vs=110311=111399#toc Repository: rL LLVM

[PATCH] D36779: [Driver] SafeStack does not need a runtime library on Fuchsia

This revision was automatically updated to reflect the committed changes. Closed by commit rL311034: [Driver] SafeStack does not need a runtime library on Fuchsia (authored by phosek). Changed prior to commit: https://reviews.llvm.org/D36779?vs=111301=111400#toc Repository: rL LLVM

[PATCH] D32613: [Driver] Update Fuchsia driver path handling

phosek updated this revision to Diff 97665. phosek added a comment. Done Repository: rL LLVM https://reviews.llvm.org/D32613 Files: lib/Driver/ToolChains/Fuchsia.cpp lib/Driver/ToolChains/Fuchsia.h test/Driver/fuchsia.cpp Index: test/Driver/fuchsia.cpp

[PATCH] D32815: [clang-tidy][CMake] Make clang-tidy usable as distribution component

phosek created this revision. phosek added a project: clang-tools-extra. Herald added subscribers: xazax.hun, mgorny. Use add_clang_tool rather than add_clang_executable to support clang-tidy as a distribution component. Repository: rL LLVM https://reviews.llvm.org/D32815 Files:

[PATCH] D32817: [CMake] Build runtimes for Fuchsia targets

phosek created this revision. Herald added a subscriber: mgorny. This relies on the multi-target runtimes build support. Repository: rL LLVM https://reviews.llvm.org/D32817 Files: cmake/caches/Fuchsia-stage2.cmake cmake/caches/Fuchsia.cmake Index: cmake/caches/Fuchsia.cmake

[PATCH] D32815: [clang-tidy][CMake] Make clang-tidy usable as distribution component

This revision was automatically updated to reflect the committed changes. Closed by commit rL302688: [clang-tidy][CMake] Make clang-tidy usable as distribution component (authored by phosek). Changed prior to commit: https://reviews.llvm.org/D32815?vs=97683=98479#toc Repository: rL LLVM

[PATCH] D32613: [Driver] Update Fuchsia driver path handling

phosek updated this revision to Diff 102271. Repository: rL LLVM https://reviews.llvm.org/D32613 Files: lib/Driver/ToolChains/Fuchsia.cpp lib/Driver/ToolChains/Fuchsia.h test/Driver/fuchsia.cpp Index: test/Driver/fuchsia.cpp

[PATCH] D32817: [CMake] Build runtimes for Fuchsia targets

phosek updated this revision to Diff 100482. Repository: rL LLVM https://reviews.llvm.org/D32817 Files: cmake/caches/Fuchsia-stage2.cmake cmake/caches/Fuchsia.cmake Index: cmake/caches/Fuchsia.cmake === ---

[PATCH] D33263: [scan-build] Patch to scan-build tool to support "--target=" flag

This revision was automatically updated to reflect the committed changes. Closed by commit rL304025: [scan-build] Patch to scan-build tool to support "--target=" flag (authored by phosek). Changed prior to commit: https://reviews.llvm.org/D33263?vs=99219=100450#toc Repository: rL LLVM

[PATCH] D34290: [Driver] Do a PATH lookup when using only program name with -no-canonical-prefixes

phosek created this revision. When -no-canonical-prefixes option is used and argv0 contains only a program name, we need to do a PATH lookup to get an executable path, otherwise the return value won't be a valid path and any subsequent uses of it (e.g. invoking -cc1) will fail with an error.

[PATCH] D34290: [Driver] Do a PATH lookup when using only program name with -no-canonical-prefixes

phosek updated this revision to Diff 102881. phosek marked an inline comment as done. phosek edited the summary of this revision. Repository: rL LLVM https://reviews.llvm.org/D34290 Files: tools/driver/driver.cpp Index: tools/driver/driver.cpp

[PATCH] D34290: [Driver] Do a PATH lookup when using only program name with -no-canonical-prefixes

phosek added inline comments. Comment at: tools/driver/driver.cpp:58 +SmallString<128> ExecutablePath(Argv0); +// Do a PATH lookup, if there are no directory components. +if (llvm::sys::path::filename(ExecutablePath) == ExecutablePath) hans wrote: >

[PATCH] D34290: [Driver] Do a PATH lookup when using only program name with -no-canonical-prefixes

This revision was automatically updated to reflect the committed changes. Closed by commit rL305600: [Driver] Do a PATH lookup if needed when using -no-canonical-prefixes (authored by phosek). Changed prior to commit: https://reviews.llvm.org/D34290?vs=102881=102894#toc Repository: rL LLVM

[PATCH] D37785: [Fuchsia] Set ENABLE_X86_RELAX_RELOCATIONS for Fuchsia builds

phosek accepted this revision. phosek added a comment. This revision is now accepted and ready to land. LGTM Repository: rL LLVM https://reviews.llvm.org/D37785 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D37723: [Driver] Fuchsia targets default to -fasynchronous-unwind-tables

phosek accepted this revision. phosek added a comment. This revision is now accepted and ready to land. LGTM Repository: rL LLVM https://reviews.llvm.org/D37723 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D37723: [Driver] Fuchsia targets default to -fasynchronous-unwind-tables

This revision was automatically updated to reflect the committed changes. Closed by commit rL312989: [Driver] Fuchsia targets default to -fasynchronous-unwind-tables (authored by phosek). Changed prior to commit: https://reviews.llvm.org/D37723?vs=114737=114746#toc Repository: rL LLVM

[PATCH] D37785: [Fuchsia] Set ENABLE_X86_RELAX_RELOCATIONS for Fuchsia builds

This revision was automatically updated to reflect the committed changes. Closed by commit rL313173: [Fuchsia] Set ENABLE_X86_RELAX_RELOCATIONS for Fuchsia builds (authored by phosek). Changed prior to commit: https://reviews.llvm.org/D37785?vs=114956=115094#toc Repository: rL LLVM

[PATCH] D37872: [libc++abi] Fix ASAN build with older compiler-rt versions.

phosek accepted this revision. phosek added a comment. This revision is now accepted and ready to land. LGTM https://reviews.llvm.org/D37872 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D37871: [ASAN] Add macro denoting availability of new `__asan_handle_no_return()` function.

phosek accepted this revision. phosek added a comment. This revision is now accepted and ready to land. LGTM https://reviews.llvm.org/D37871 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D39017: [CMake] Build Fuchsia toolchain as -O3

phosek created this revision. Herald added a subscriber: mgorny. Repository: rL LLVM https://reviews.llvm.org/D39017 Files: cmake/caches/Fuchsia-stage2.cmake Index: cmake/caches/Fuchsia-stage2.cmake === ---

[PATCH] D39017: [CMake] Build Fuchsia toolchain as -O3

This revision was automatically updated to reflect the committed changes. Closed by commit rL316053: [CMake] Build Fuchsia toolchain as -O3 (authored by phosek). Changed prior to commit: https://reviews.llvm.org/D39017?vs=119372=119417#toc Repository: rL LLVM

[PATCH] D39176: [Driver] Use ld.lld directly for Fuchsia rather than passing flavor

phosek updated this revision to Diff 119923. Repository: rL LLVM https://reviews.llvm.org/D39176 Files: lib/Driver/ToolChains/Fuchsia.cpp lib/Driver/ToolChains/Fuchsia.h test/Driver/fuchsia.c test/Driver/fuchsia.cpp Index: test/Driver/fuchsia.cpp

[PATCH] D39176: [Driver] Use ld.lld directly for Fuchsia rather than passing flavor

This revision was automatically updated to reflect the committed changes. Closed by commit rL316379: [Driver] Use ld.lld directly for Fuchsia rather than passing flavor (authored by phosek). Changed prior to commit: https://reviews.llvm.org/D39176?vs=119923=119938#toc Repository: rL LLVM

[PATCH] D39176: [Driver] Use ld.lld directly for Fuchsia rather than passing flavor

phosek created this revision. phosek added a project: clang. Passing a flavor to LLD requires command line argument, but if these are being passed through a response file, this will fail because LLD needs to know which driver to use before processing the response file. Use ld.lld directly instead

[PATCH] D39821: Add CLANG_DEFAULT_OBJCOPY to allow Clang to use llvm-objcopy for dwarf fission

phosek accepted this revision. phosek added a comment. This revision is now accepted and ready to land. LGTM Repository: rL LLVM https://reviews.llvm.org/D39821 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D40257: [CMake] Use LIST_SEPARATOR rather than escaping in ExternalProject_Add

phosek added a comment. Can you please take a look at this one as well (this one is Clang, the other one was LLVM)? Repository: rL LLVM https://reviews.llvm.org/D40257 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D40513: [libcxx] Support the use of compiler-rt in lit tests

phosek created this revision. Herald added subscribers: mgorny, dberris. Don't link tests against libgcc when compiler-rt is being used. Repository: rL LLVM https://reviews.llvm.org/D40513 Files: test/CMakeLists.txt test/lit.site.cfg.in utils/libcxx/test/target_info.py Index:

[PATCH] D40329: [CMake][Fuchsia] Disable terminfo database in Fuchsia toolchain

This revision was automatically updated to reflect the committed changes. Closed by commit rL319088: [CMake][Fuchsia] Disable terminfo database in Fuchsia toolchain (authored by phosek). Changed prior to commit: https://reviews.llvm.org/D40329?vs=123875=124441#toc Repository: rL LLVM

[PATCH] D40637: [CMake] Support runtimes and monorepo layouts when looking for libc++

phosek created this revision. Herald added subscribers: Sanitizers, llvm-commits, mgorny, kubamracek. Herald added a reviewer: EricWF. This also slightly refactors the code that's checking the directory presence which allows eliminating one unnecessary variable. Repository: rCRT Compiler

[PATCH] D40280: [CMake][libcxx] Include AddLLVM needed for tests in the right context

This revision was automatically updated to reflect the committed changes. Closed by commit rL319515: Include AddLLVM needed for tests in the right context (authored by phosek). Changed prior to commit: https://reviews.llvm.org/D40280?vs=123711=125072#toc Repository: rL LLVM

  1   2   3   4   5   6   7   8   9   10   >