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

haowei created this revision. Herald added a subscriber: whisperity. I noticed that when I use "-analyze-config" option in scan-build-py, it behaves differently from original perl based scan-build. For example, command: $ scan-build -analyzer-config ipa=basic-inlining make Will work without an

[PATCH] D100139: [ifs][elfabi] Merge llvm-ifs/elfabi tools

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG6103fdfab4e2: [ifs][elfabi] Merge llvm-ifs/elfabi tools (authored by haowei). Changed prior to commit: https://reviews.llvm.org/D100139?vs=353507&

[PATCH] D119479: [clang][extract-api] Add global record support

haowei added a comment. We are seeing test failures after this patch is landed: Clang :: SymbolGraph/global_record.c Script: -- : 'RUN: at line 1'; rm -rf /b/s/w/ir/x/w/staging/llvm_build/tools/clang/test/SymbolGraph/Output/global_record.c.tmp : 'RUN: at line 2'; split-file /b/s/w

[PATCH] D119479: [clang][extract-api] Add global record support

haowei added a comment. In D119479#3387880 , @zixuw wrote: > @haowei I got a quick fix of completely purging the generator output in the > test case. Could you take a look to have a sanity check before I land the fix > on main? > > diff --git a/clang/

[PATCH] D113620: Skip exception cleanups when the innermost scope is EHTerminateScope.

haowei added a comment. Hi, we are seeing clang Windows build breakages after this change was relanded error message: Assertion failed: isa(Val) && "cast() argument of incompatible type!", file C:\b\s\w\ir\x\w\llvm-llvm-project\llvm\include\llvm/Support/Casting.h, line 262 PLEASE submit a

[PATCH] D36022: [analyzer] Add handle misuse analysis to MagentaHandleChecker

haowei created this revision. Herald added a subscriber: xazax.hun. This commits add actual code for analyzing magenta handle misuse issues to MagentaHandleChecker, which is introduced in https://reviews.llvm.org/D35968. It supports magenta syscalls that acquires/release/use handle through point

[PATCH] D36023: [analyzer] Add array support for MagentaHandleChecker

haowei created this revision. Herald added a subscriber: xazax.hun. This commit adds support for syscalls that acquire/release handles in an array for MagentaHandleChecker introduced in https://reviews.llvm.org/D35968 and https://reviews.llvm.org/D36022. Most magenta handle related syscalls wil

[PATCH] D36024: [analyzer] Improved bug reporting in MagentaHandleChecker

haowei created this revision. Herald added a subscriber: xazax.hun. This commit improves the bug reporting of MagentaHandleChecker introduced in https://reviews.llvm.org/D35968 , https://reviews.llvm.org/D36022 and https://reviews.llvm.org/D36023. After this commit, the allocation and release s

[PATCH] D36022: [analyzer] Add handle misuse analysis to MagentaHandleChecker

haowei updated this revision to Diff 109237. haowei added a comment. Add 1 line fix for the constraint on allocated handle in function allocateSingleHandle at line 647 https://reviews.llvm.org/D36022 Files: lib/StaticAnalyzer/Checkers/MagentaHandleChecker.cpp test/Analysis/mxhandle.c Inde

[PATCH] D36251: [analyzer] Suppress warning when bug path contains noreturn function or return from main in MagentaHandleChecker

haowei created this revision. Herald added a subscriber: xazax.hun. This patch will suppress handle leak warnings if the path of a bug reported by MagentaHandleChecker will reach a no return function or will reach the end of main function. As these two scenarios mean program crash and the leaked

[PATCH] D36475: [analyzer] Add "create_sink" annotation support to MagentaHandleChecker

haowei created this revision. Herald added a subscriber: xazax.hun. This patch adds "mx_create_sink" annotation support to MagentaHandleChecker to address the false positives found in Magenta unit tests. After this patch, when a call to a function contains this annotation, MagentaHandleChecker w

[PATCH] D36251: [analyzer] Suppress warning when bug path contains noreturn function or return from main in MagentaHandleChecker

haowei added inline comments. Comment at: lib/StaticAnalyzer/Checkers/MagentaHandleChecker.cpp:483-498 +void MagentaHandleChecker::checkPreStmt(const ReturnStmt *RS, +CheckerContext &Ctx) const { + ProgramStateRef State = Ctx.getState(); +

[PATCH] D90275: [clang][IR] Add support for leaf attribute

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG7c0e3a77bc43: [clang][IR] Add support for leaf attribute (authored by gulfem, committed by haowei). Repository: rG LLVM Github Monorepo CHANGES S

[PATCH] D90362: scan-build supprot relative 'file' in cdb.

haowei closed this revision. haowei added a comment. Landed in https://github.com/llvm/llvm-project/commit/d93287cac89fd50a105ac4a59c079884b8e53e49 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D90362/new/ https://reviews.llvm.org/D90362 _

[PATCH] D66324: clang-misexpect: Profile Guided Validation of Performance Annotations in LLVM

haowei added a comment. In D66324#2394687 , @lebedev.ri wrote: > In D66324#2336555 , @lebedev.ri > wrote: > >> In D66324#2336186 , @phosek wrote: >> >>> I apologize for the

[PATCH] D91223: Support struct annotations in FuchsiaHandleChecker.

This revision was automatically updated to reflect the committed changes. Closed by commit rG914f6c4ff8a4: [StaticAnalyzer] Support struct annotations in FuchsiaHandleChecker (authored by haowei). Herald added a project: clang. Herald added a subscriber: cfe-commits. Changed prior to commit: ht

[PATCH] D91223: Support struct annotations in FuchsiaHandleChecker.

haowei added a comment. In D91223#2415137 , @MaskRay wrote: > @haowei https://llvm.org/docs/DeveloperPolicy.html#commit-messages Please use > git commit --amend --author for future patches where the majority of the work > is done by others. Thanks for y

[PATCH] D92444: [CMake][Fuchsia] Install llvm-elfabi

haowei accepted this revision. haowei 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/D92444/new/ https://reviews.llvm.org/D92444 ___ c

[PATCH] D91902: [analyzer] Ignore annotations if func is inlined.

This revision was automatically updated to reflect the committed changes. Closed by commit rG3ce78f54edcf: [analyzer] Ignore annotations if func is inlined. (authored by aabbaabb, committed by haowei). Herald added a project: clang. Herald added a subscriber: cfe-commits. Changed prior to commit:

[PATCH] D102693: Do not create LLVM IR `constant`s for objects with dynamic initialisation

haowei added a comment. This CL breaks our mac builders as well. Could you either fix the test or revert the change please? Error message: TEST 'Clang :: CodeGenCXX/const-dynamic-init.cpp' FAILED Script: -- : 'RUN: at line 1'; /opt/staging/ll

[PATCH] D103218: [Fuchsia][CMake] Add missing include path.

haowei created this revision. haowei added reviewers: phosek, mcgrathr, leonardchan, gulfem. Herald added subscribers: cryptoad, mgorny. haowei requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. This patch adds include path for missing header

[PATCH] D103218: [Fuchsia][CMake] Add missing include path.

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG0ce58c52d50b: [Fuchsia][CMake] Add missing include path. (authored by haowei). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION ht

[PATCH] D94251: [scan-build-py] Update scan-build-py to allow outputing as SARIF.

This revision was automatically updated to reflect the committed changes. Closed by commit rGd72859ffa237: [scan-build-py] Update scan-build-py to allow outputing as SARIF (authored by Daniel Hwang , committed by haowei). Herald added a project: clang. Herald added a subscriber:

[PATCH] D110668: [clang-cl] Accept `#pragma warning(disable : N)` for some N

haowei added a comment. We are seeing a series of weird errors in our windows clang builder after this patch landed: [1/828] Building CXX object compiler-rt\lib\sanitizer_common\CMakeFiles\RTSanitizerCommonNoTermination.x86_64.dir\sanitizer_solaris.cpp.obj FAILED: compiler-rt/lib/sanitizer

[PATCH] D96389: [analyzer] Update static analyzer to be support sarif-html

This revision was automatically updated to reflect the committed changes. Closed by commit rG2407eb08a574: [analyzer] Update static analyzer to be support sarif-html (authored by Daniel Hwang , committed by haowei). Herald added a project: clang. Herald added a subscriber: cfe-c

[PATCH] D99810: [ifs] Prepare llvm-ifs for elfabi/ifs merging.

haowei marked an inline comment as done. haowei added inline comments. Comment at: llvm/include/llvm/InterfaceStub/ELFObjHandler.h:25 namespace elfabi { phosek wrote: > Shouldn't this be in namespace `ifs`? This was renamed to ifs in D100139. Rename it here

[PATCH] D100139: [ifs][elfabi] Merge llvm-ifs/elfabi tools

haowei updated this revision to Diff 353507. haowei marked 3 inline comments as done. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D100139/new/ https://reviews.llvm.org/D100139 Files: clang/cmake/caches/Fuchsia-stage2.cmake clang/lib/Driver/Tool

[PATCH] D74436: Change clang option -ffp-model=precise to select ffp-contract=on

haowei added a comment. We are seeing float related unit tests failing in Fuchsia after this patch. What flags should we add to our build to match llvm's previous float point behavior? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D74436/new/ http

[PATCH] D74436: Change clang option -ffp-model=precise to select ffp-contract=on

haowei added a comment. Can we revert this patch please? If I understood correctly, this patch changed the default behavior of clang when generating code for float point arithmetic. After this patch `-ffp-contract` was set to on if this flag was not specified. In theory this makes clang use FMA

[PATCH] D74436: Change clang option -ffp-model=precise to select ffp-contract=on

haowei added a comment. In D74436#2926385 , @tstellar wrote: > Can someone file a bug for this and put release-13.0.0 in the blocks field so > we can track it? Filed https://bugs.llvm.org/show_bug.cgi?id=51346 . Repository: rG LLVM Github Monorepo C

[PATCH] D128619: [Clang] Implement P0848 (Conditionally Trivial Special Member Functions)

haowei added subscribers: haowei, phosek, mcgrathr. haowei added a comment. Hi, We are seeing build failures on Fuchsia build bots after this change was landed. and we suspect this patch causes clang to miscompile certain code. I filed bug https://github.com/llvm/llvm-project/issues/57351 and i

[PATCH] D131714: [compiler-rt][builtins] Add compiler flags to catch potential errors that can lead to security vulnerabilities

haowei added a comment. We are also saw failures on Fuchsia builders when using ToT clang: [31281/267743] CXX host_x64/obj/src/lib/llvm-profdata/llvm-profdata-for-test.llvm-profdata.cc.o FAILED: host_x64/obj/src/lib/llvm-profdata/llvm-profdata-for-test.llvm-profdata.cc.o ../../../recipe_c

[PATCH] D131714: [compiler-rt][builtins] Add compiler flags to catch potential errors that can lead to security vulnerabilities

haowei added subscribers: mcgrathr, phosek, gulfem. haowei added a comment. After looking into the code, I think I should point out that `compiler-rt/include/profile/InstrProfData.inc` is a published header and its API details cannot be changed trivially without coordination with external users

[PATCH] D134235: [clang-doc] Clean up *Info constructors.

haowei added a comment. I am a bit puzzled by the intent of cleaning up the constructors in Info object. I didn't see it as an issue to have multiple constructors, which is a common practice. After this clean up It doesn't make it easier to use these objects, in fact, in `Serialize.cpp`, you ha

[PATCH] D134235: [clang-doc] Clean up *Info constructors.

haowei added a comment. @brettw Thanks for the clarification. I was oncall for buildgardener last week and got quite busy due to breakages. Thanks for your clarification, that is really helpful to understand the your intent. While I still don't see the way these constructors are implemented is

[PATCH] D134235: [clang-doc] Clean up *Info constructors.

haowei added inline comments. Comment at: clang-tools-extra/clang-doc/BitcodeReader.cpp:336 Reference *I, FieldId &F) { switch (ID) { case REFERENCE_USR: This block ID came from L582, which read from a bitcode file when `BI_REFEREN

[PATCH] D134235: [clang-doc] Clean up *Info constructors.

haowei accepted this revision. haowei added a comment. This revision is now accepted and ready to land. LGTM @paulkirth Do you have more comments? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D134235/new/ https://reviews.llvm.org/D134235 ___

[PATCH] D134371: [clang-doc] Add typedef/using information.

haowei added a comment. In D134371#3819605 , @nemanjai wrote: > This causes failures with `-Werror` such as: > https://lab.llvm.org/buildbot/#/builders/57/builds/22322 > Please fix or revert. Thanks for reporting this. I reverted this change. Reposito

[PATCH] D141448: [CMake][Fuchsia] Include llvm-ml in Fuchsia toolchain

haowei created this revision. haowei added a reviewer: phosek. Herald added a subscriber: abrachet. Herald added a project: All. haowei requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. This change includes llvm-ml tool in Fuchsia toolchain.

[PATCH] D141448: [CMake][Fuchsia] Include llvm-ml in Fuchsia toolchain

This revision was automatically updated to reflect the committed changes. Closed by commit rGbc3edb627201: [CMake][Fuchsia] Include llvm-ml in Fuchsia toolchain (authored by haowei). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D141448/new/ https://

[PATCH] D137473: [vfs] Allow root paths relative to the directory of the vfsoverlay YAML file

haowei created this revision. haowei added reviewers: phosek, abrachet. Herald added a subscriber: hiraditya. Herald added a project: All. haowei requested review of this revision. Herald added projects: clang, LLVM. Herald added subscribers: llvm-commits, cfe-commits. VFS overlay file allow using

[PATCH] D137473: [vfs] Allow root paths relative to the directory of the vfsoverlay YAML file

haowei added reviewers: bnbarham, keith. haowei added a comment. Please take a look. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D137473/new/ https://reviews.llvm.org/D137473 ___ cfe-commits mailing lis

[PATCH] D137473: [vfs] Allow root paths relative to the directory of the vfsoverlay YAML file

haowei updated this revision to Diff 474115. haowei marked 2 inline comments as done. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D137473/new/ https://reviews.llvm.org/D137473 Files: clang/test/VFS/Inputs/root-relative-overlay.yaml clang/test/V

[PATCH] D137473: [vfs] Allow root paths relative to the directory of the vfsoverlay YAML file

haowei added inline comments. Comment at: clang/test/VFS/Inputs/root-relative-overlay.yaml:4 + 'case-sensitive': false, + 'overlay-relative': true, + 'root-relative': 'yaml-dir', bnbarham wrote: > bnbarham wrote: > > I'd prefer a test without `overlay-relative

[PATCH] D137473: [vfs] Allow root paths relative to the directory of the vfsoverlay YAML file

haowei updated this revision to Diff 474391. haowei added a comment. Add additional unit test. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D137473/new/ https://reviews.llvm.org/D137473 Files: clang/test/VFS/Inputs/root-relative-overlay.yaml c

[PATCH] D137473: [vfs] Allow root paths relative to the directory of the vfsoverlay YAML file

haowei added inline comments. Comment at: clang/test/VFS/Inputs/root-relative-overlay.yaml:4 + 'case-sensitive': false, + 'overlay-relative': true, + 'root-relative': 'yaml-dir', haowei wrote: > haowei wrote: > > bnbarham wrote: > > > bnbarham wrote: > > > > I

[PATCH] D137473: [vfs] Allow root paths relative to the directory of the vfsoverlay YAML file

haowei updated this revision to Diff 475347. haowei added a comment. Unit test failures under Windows should be fixed now. Root->name will also use base FS's current directory instead of process current directory now. Please take a look. Repository: rG LLVM Github Monorepo CHANGES SINCE LAS

[PATCH] D137473: [vfs] Allow root paths relative to the directory of the vfsoverlay YAML file

haowei updated this revision to Diff 475574. haowei added a comment. I have to revert back to use process cwd instead of base fs cwd. There are a few tests relied the behavior of using process cwd that I need to take a closer look. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTI

[PATCH] D137473: [vfs] Allow root paths relative to the directory of the vfsoverlay YAML file

haowei updated this revision to Diff 475591. haowei added a comment. Address review comments Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D137473/new/ https://reviews.llvm.org/D137473 Files: clang/test/VFS/Inputs/root-relative-overlay.yaml cla

[PATCH] D137473: [vfs] Allow root paths relative to the directory of the vfsoverlay YAML file

haowei marked 2 inline comments as done. haowei added inline comments. Comment at: llvm/include/llvm/Support/VirtualFileSystem.h:1010 + std::error_code makeAbsolute(StringRef WorkingDir, + SmallVectorImpl &Path) const; bnbarham wr

[PATCH] D137473: [vfs] Allow root paths relative to the directory of the vfsoverlay YAML file

haowei updated this revision to Diff 475884. haowei marked 2 inline comments as done. haowei added a comment. Fixing Windows test failures. overlay-relative flag will always using native path separator, therefore, it needs a separate base FS and OverlayYAML file setup on Windows. This diff adds

[PATCH] D143025: [Fuchsia] Add llvm-mt and llvm-rc to clang bootstrap dependency

haowei created this revision. haowei added a reviewer: phosek. Herald added a subscriber: abrachet. Herald added a project: All. haowei requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. llvm-mt and llvm-rc tools are capable of building clang

[PATCH] D141738: Add initial support for cross compile Windows runtimes under Linux when building Fuchsia clang toolchain

haowei updated this revision to Diff 493766. haowei added a comment. Correct dependency issues on llvm-mt Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D141738/new/ https://reviews.llvm.org/D141738 Files: clang/cmake/caches/Fuchsia-stage2.cmake

[PATCH] D143025: [Fuchsia] Add llvm-mt and llvm-rc to clang bootstrap dependency

haowei updated this revision to Diff 494364. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D143025/new/ https://reviews.llvm.org/D143025 Files: clang/cmake/caches/Fuchsia.cmake Index: clang/cmake/caches/Fuchsia.cmake ===

[PATCH] D141738: Add initial support for cross compile Windows runtimes under Linux when building Fuchsia clang toolchain

haowei added inline comments. Comment at: clang/cmake/caches/Fuchsia-stage2.cmake:95 +# and remove these libpath flags. +-libpath:"${LLVM_WINSYSROOT}/VC/Tools/MSVC/14.34.31933/lib/x64" +-libpath:"${LLVM_WINSYSROOT}/VC/Tools/MSVC/14.34.31933/atlmfc/lib/x64"

[PATCH] D143288: [Fuchsia] Simplified the stage2 build setup

haowei created this revision. haowei added a reviewer: phosek. Herald added a subscriber: abrachet. Herald added a project: All. haowei requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. This patch simplified the BOOTSTRAP_ flags, allowing the

[PATCH] D143025: [Fuchsia] Add llvm-mt and llvm-rc to clang bootstrap dependency

haowei updated this revision to Diff 494755. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D143025/new/ https://reviews.llvm.org/D143025 Files: clang/CMakeLists.txt Index: clang/CMakeLists.txt =

[PATCH] D143025: [Fuchsia] Add llvm-mt and llvm-rc to clang bootstrap dependency

haowei marked 2 inline comments as done. haowei added inline comments. Comment at: clang/cmake/caches/Fuchsia.cmake:170-171 runtimes + llvm-rc + llvm-mt CACHE STRING "") phosek wrote: > Could we do this in > https://github.com/llvm/llvm-project/blob/98f0

[PATCH] D143288: [Fuchsia] Simplified the stage2 build setup

This revision was automatically updated to reflect the committed changes. Closed by commit rG630266aed342: [Fuchsia] Simplified the stage2 build setup (authored by haowei). Changed prior to commit: https://reviews.llvm.org/D143288?vs=494694&id=495216#toc Repository: rG LLVM Github Monorepo

[PATCH] D143025: [Clang] Add llvm-mt and llvm-rc to Clang bootstrap dependency

haowei updated this revision to Diff 495221. haowei marked an inline comment as done. haowei retitled this revision from "[Fuchsia] Add llvm-mt and llvm-rc to clang bootstrap dependency" to "[Clang] Add llvm-mt and llvm-rc to Clang bootstrap dependency". haowei edited the summary of this revision

[PATCH] D143025: [Clang] Add llvm-mt and llvm-rc to Clang bootstrap dependency

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGdc1c8917afd3: [Clang] Add llvm-mt and llvm-rc to Clang bootstrap dependency (authored by haowei). Repository: rG LLVM Github Monorepo CHANGES SIN

[PATCH] D141738: Add initial support for cross compile Windows runtimes under Linux when building Fuchsia clang toolchain

This revision was automatically updated to reflect the committed changes. Closed by commit rG5fcdab04c250: [Fuchsia] Build windows runtimes using cross compilation on Linux (authored by haowei). Changed prior to commit: https://reviews.llvm.org/D141738?vs=493766&id=495281#toc Repository: rG

[PATCH] D141738: [WIP] Add initial support for cross compile Windows runtimes under Linux when building Fuchsia clang toolchain

haowei created this revision. haowei added a reviewer: phosek. Herald added a subscriber: abrachet. Herald added a project: All. haowei requested review of this revision. Herald added projects: clang, LLVM. Herald added subscribers: llvm-commits, cfe-commits. Work in progress. Initial work on bui

[PATCH] D141738: [WIP] Add initial support for cross compile Windows runtimes under Linux when building Fuchsia clang toolchain

haowei updated this revision to Diff 489177. haowei added a comment. Correct C, CXX flag issue. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D141738/new/ https://reviews.llvm.org/D141738 Files: clang/cmake/caches/Fuchsia-stage2.cmake llvm/cmak

[PATCH] D141738: Add initial support for cross compile Windows runtimes under Linux when building Fuchsia clang toolchain

haowei updated this revision to Diff 490679. haowei retitled this revision from "[WIP] Add initial support for cross compile Windows runtimes under Linux when building Fuchsia clang toolchain" to "Add initial support for cross compile Windows runtimes under Linux when building Fuchsia clang tool

[PATCH] D141738: Add initial support for cross compile Windows runtimes under Linux when building Fuchsia clang toolchain

haowei marked 2 inline comments as done. haowei added inline comments. Comment at: clang/cmake/caches/Fuchsia-stage2.cmake:73 + +if(WIN32 OR (LINUX AND WINDOWS_SDK_DIR)) +#if(WINDOWS_SDK_DIR) phosek wrote: > I'd drop this part altogether, let's build these anytim

[PATCH] D141738: Add initial support for cross compile Windows runtimes under Linux when building Fuchsia clang toolchain

haowei updated this revision to Diff 490921. haowei added a comment. Correct an issue llvm-rc and llvm-mt are not part of `TOOLCHAIN_TOOLS` Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D141738/new/ https://reviews.llvm.org/D141738 Files: clang/c

[PATCH] D141738: Add initial support for cross compile Windows runtimes under Linux when building Fuchsia clang toolchain

haowei updated this revision to Diff 490994. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D141738/new/ https://reviews.llvm.org/D141738 Files: clang/cmake/caches/Fuchsia-stage2.cmake llvm/cmake/modules/LLVMExternalProjectUtils.cmake Index: llv

[PATCH] D141738: Add initial support for cross compile Windows runtimes under Linux when building Fuchsia clang toolchain

haowei added a comment. Thanks for the review. I will land the change once Fuchsia clang builders are green. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D141738/new/ https://reviews.llvm.org/D141738 __

[PATCH] D141738: Add initial support for cross compile Windows runtimes under Linux when building Fuchsia clang toolchain

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG20bb7658f53b: [Fuchsia] Build windows runtimes using cross compilation on Linux (authored by haowei). Changed prior to commit: https://reviews.llv

[PATCH] D141738: Add initial support for cross compile Windows runtimes under Linux when building Fuchsia clang toolchain

haowei added inline comments. Comment at: clang/cmake/caches/Fuchsia-stage2.cmake:95 +# and remove these libpath flags. +-libpath:"${LLVM_WINSYSROOT}/VC/Tools/MSVC/14.34.31933/lib/x64" +-libpath:"${LLVM_WINSYSROOT}/VC/Tools/MSVC/14.34.31933/atlmfc/lib/x64"

[PATCH] D126192: [Driver] Improve linking options for target AVR

haowei added a comment. We are seeing Clang driver test failures after this patch. On Fuchsia's mac clang builder, `Clang :: Driver/avr-ld.c` and `Clang :: Driver/avr-toolchain.c` are failing. Failed build task: https://ci.chromium.org/ui/p/fuchsia/builders/toolchain.ci/clang-mac-x64/b8811424

[PATCH] D126192: [Driver] Improve linking options for target AVR

haowei added a comment. I already reverted the change in https://github.com/llvm/llvm-project/commit/7fae15f9251d3b392058e2fc84898b53619d36ad . It is likely the breakage is caused by the fact that Fuchsia's Clang by default use `lld`, see `clang/cmake/caches/Fuchsia-stage2.cmake:set(CLANG_DEFA

[PATCH] D127814: [Driver] Force avr-ld to be default linker for target AVR

haowei added a comment. I tested this patch on top of https://github.com/llvm/llvm-project/commit/3b6e166999bb83197c8b70468c2d371c58cf0c25 (reverted in https://github.com/llvm/llvm-project/commit/7fae15f9251d3b392058e2fc84898b53619d36ad) and Clang driver tests now pass without issue. Reposit

[PATCH] D126192: [Driver] Improve linking options for target AVR

haowei added a comment. Emmm, I applied your latest patch on top of main(6e02e27536b9de25a651cfc9c2966ce471169355 ) but I still get a test failure on `Clang :: Driver/avr-toolchain.c`. `Clang :: Driver/avr-ld.c` no longer fai

[PATCH] D126192: [Driver] Improve linking options for target AVR

haowei added a comment. In D126192#3583915 , @benshi001 wrote: > It seems the file > `/mnt/nvme_sec/SRC/llvm-project/clang/test/Driver/Inputs/basic_avr_tree/usr/bin/ld.lld` > is missing on your machine, which I have newly created in current patch. That

[PATCH] D126192: [Driver] Improve linking options for target AVR

haowei added a comment. I tested this patch on our windows builder and it passed as well. This patch LGTM. (I cannot click accept as the this review is already closed) CHANGES SINCE LAST ACTION https://reviews.llvm.org/D126192/new/ https://reviews.llvm.org/D126192 __

[PATCH] D145214: [TSAN] add support for riscv64

haowei added a comment. Hi, I believe you forgot to add `tsan_rtl_riscv64.S` to your patch when you land it. We are seeing build errors on our bots with following messages: -- Configuring done (12.3s) CMake Error at /b/s/w/ir/x/w/llvm-llvm-project/compiler-rt/cmake/Modules/AddCompilerRT.cm

[PATCH] D153557: [clang][ExtractAPI] Add support for C++ classes

haowei added a comment. Hi, we are seeing a test error on `Clang :: ExtractAPI/constructor_destructor.cpp` after this patch was landed. Error message: Script: -- : 'RUN: at line 1'; rm -rf /b/s/w/ir/x/w/llvm_build/tools/clang/test/ExtractAPI/Output/constructor_destructor.cpp.tmp : 'R

[PATCH] D153557: [clang][ExtractAPI] Add support for C++ classes

haowei added a comment. Thanks for reverting. I wasn't doing any changes to this when you were asking though, but looking at build log, it looks like the assertion error disappeared after f4de606ef271fe362f03d30c53a850f9877ec238

[PATCH] D153557: [clang][ExtractAPI] Add support for C++ classes

haowei added a comment. FYI, this patch fails the Windows bots in a different way. (you can see the error triggered in the pre-merge build in https://buildkite.com/llvm-project/premerge-checks/builds/161603#018909c4-fe90-4de6-91a3-d251dc5ada8f) The error message from our builders are: Script

[PATCH] D158948: [clang][ASTImporter] Add import of type-related nodes

haowei added a comment. Hi I am working on updating GoogleTest in LLVM to v1.14.0 and during my local testing, I noticed that this patch added 2 new tests `ImportMatrixType` and `ImportOpenCLPipe` through `TEST_P` calls, however, both tests were not instantiated by `INSTANTIATE_TEST_SUITE_P` so

[PATCH] D158948: [clang][ASTImporter] Add import of type-related nodes

haowei added a comment. In D158948#4642475 , @danix800 wrote: > In D158948#4642422 , @haowei wrote: > >> Looking at >> https://github.com/llvm/llvm-project/blob/37a20cc68f545647e614c5ba4ae311dc3fd277e9/clang/lib/

[PATCH] D155241: [Fuchsia] Skip building debuginfod if httplib_ROOT is not specified

haowei created this revision. haowei added reviewers: phosek, mysterymath. Herald added a subscriber: abrachet. Herald added a project: All. haowei requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. This patch skipds building debuginfod if htt

[PATCH] D137473: [vfs] Allow root paths relative to the directory of the vfsoverlay YAML file

haowei added inline comments. Comment at: llvm/lib/Support/VirtualFileSystem.cpp:1922 + EC = FS->makeAbsolute(FullPath, Name); + Name = canonicalize(Name); +} else { bnbarham wrote: > Why the canonicalization? If the Name here somehow ha

[PATCH] D145010: [Fuchsia] Enable LLVM_USE_RELATIVE_PATHS_IN_FILES in bootstrap

haowei created this revision. haowei added reviewers: phosek, abrachet. Herald added a project: All. haowei requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. This patch enables LLVM_USE_RELATIVE_PATHS_IN_FILES when building the bootstrap too

[PATCH] D145010: [Fuchsia] Enable LLVM_USE_RELATIVE_PATHS_IN_FILES in bootstrap

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGa7bb9a363923: [Fuchsia] Enable LLVM_USE_RELATIVE_PATHS_IN_FILES in bootstrap (authored by haowei). Repository: rG LLVM Github Monorepo CHANGES SI

[PATCH] D144603: Add option to disable compiler launcher on external projects

haowei added a comment. In D144603#4162048 , @smeenai wrote: > Hmm, what cache key does ccache use for compilers? Is it the `--version` > output string, the path, or some combination? If the path is involved then I > don't see any value in using ccache

[PATCH] D144603: Add option to disable compiler launcher on external projects

haowei added a comment. In D144603#4162223 , @phosek wrote: > I agree, in that case let's remove > `C_COMPILER_LAUNCHER;CXX_COMPILER_LAUNCHER` from the list of default > passthrough variables which seems like a reasonable default, and provide > `CLANG_

[PATCH] D144603: Add option to disable compiler launcher on external projects

haowei updated this revision to Diff 501670. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D144603/new/ https://reviews.llvm.org/D144603 Files: clang/CMakeLists.txt clang/cmake/caches/Fuchsia-stage2.cmake clang/cmake/caches/Fuchsia.cmake llvm

[PATCH] D144603: Disable compiler launcher on external projects and multi stage clang

haowei updated this revision to Diff 502841. haowei retitled this revision from "Add option to disable compiler launcher on external projects" to "Disable compiler launcher on external projects and multi stage clang". haowei edited the summary of this revision. Repository: rG LLVM Github Monor

[PATCH] D145449: [Fuchsia] Add LLDB options to stage 1 cmake.

haowei added inline comments. Comment at: clang/cmake/caches/Fuchsia.cmake:167 +if(FUCHSIA_ENABLE_LLDB) + list(APPEND _FUCHSIA_ENABLE_PROJECTS lldb) +endif() You probably need a `string(REPLACE ";" "|" value "${_FUCHSIA_ENABLE_PROJECTS}")` after append a value.

[PATCH] D145449: [Fuchsia] Add LLDB options to stage 1 cmake.

haowei accepted this revision. haowei added a comment. This revision is now accepted and ready to land. LGTM, but please wait @phosek to approve it. Comment at: clang/cmake/caches/Fuchsia.cmake:167 +if(FUCHSIA_ENABLE_LLDB) + list(APPEND _FUCHSIA_ENABLE_PROJECTS lldb) +endif()

[PATCH] D153931: [Fuchsia] Enable libcxx filesystem on Windows build.

haowei created this revision. haowei added reviewers: phosek, Prabhuk. Herald added a subscriber: abrachet. Herald added a project: All. haowei requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. This patch enables the libcxx filesystem when bu

[PATCH] D153931: [Fuchsia] Enable libcxx filesystem on Windows build.

haowei updated this revision to Diff 535194. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D153931/new/ https://reviews.llvm.org/D153931 Files: clang/cmake/caches/Fuchsia-stage2.cmake Index: clang/cmake/caches/Fuchsia-stage2.cmake ==

[PATCH] D153931: [Fuchsia] Enable libcxx filesystem on Windows build.

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGfa872aa221e2: [Fuchsia] Enable libcxx filesystem on Windows build. (authored by haowei). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST A

[PATCH] D146987: [Assignment Tracking] Enable by default

haowei added a comment. This patch (which enables assignment tracking) and D147312 breaks llvm runtime build for `runtimes-armv7-unknown-linux-gnueabihf` Error message: FAILED: libcxx/src/CMakeFiles/cxx_static.dir/charconv.cpp.o /b/s/w/ir/cache/goma/clie

[PATCH] D148383: [driver] Enable response file when using lld on baremetal targets

haowei created this revision. haowei added reviewers: mcgrathr, phosek. Herald added a subscriber: abidh. Herald added a project: All. haowei requested review of this revision. Herald added subscribers: cfe-commits, MaskRay. Herald added a project: clang. This patch enables response file support w

[PATCH] D150939: [CMake] Remove BOLT from Fuchsia toolchain

haowei added a comment. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D150939/new/ https://reviews.llvm.org/D150939 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-

[PATCH] D143300: [randstruct] Don't allow implicit forward decl to stop struct randomization

haowei added a comment. FYI, this patch looks like was pushed into a new "Main" branch instead of the actual "main" branch. It causes git checkout failures on Windows, see error message: warning: encountered old-style '/ssl/certs/ca-bundle.crt' that should be '%(prefix)//ssl/certs/ca-bundle.c

  1   2   >