[PATCH] D36555: Move x86-specific sources to x86-specific source lists.

aheejin added a comment. Looks OK to me, but I haven't worked on this part of code either, so I think someone else should approve it. https://reviews.llvm.org/D36555 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D36764: The following functions and tests work fine for powerpc64, so enable them.

aheejin resigned from this revision. aheejin added a comment. I don't have enough context or knowledge on this code base. Sorry! https://reviews.llvm.org/D36764 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D39455: [CodeGen] Add initial support for union members in TBAA

aheejin added a comment. Once you confirm the bug, could you possibly revert the patch? Repository: rL LLVM https://reviews.llvm.org/D39455 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D39455: [CodeGen] Add initial support for union members in TBAA

aheejin added a comment. Thank you for confirming and reverting! Repository: rL LLVM https://reviews.llvm.org/D39455 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D39455: [CodeGen] Add initial support for union members in TBAA

aheejin added a comment. After this patch, a couple of gcc torture tests started to fail. Could you possibly look at this? https://bugs.llvm.org/show_bug.cgi?id=35503 Repository: rL LLVM https://reviews.llvm.org/D39455 ___ cfe-commits mailing

[PATCH] D44931: [WebAssembly] Use Windows EH instructions for Wasm EH

aheejin updated this revision to Diff 146076. aheejin added a comment. - getMSVCDispatchBlock -> getFuncletEHDispatchBlock Repository: rC Clang https://reviews.llvm.org/D44931 Files: lib/CodeGen/CGCXXABI.h lib/CodeGen/CGCleanup.cpp lib/CodeGen/CGCleanup.h lib/CodeGen/CGException.cpp

[PATCH] D44931: [WebAssembly] Use Windows EH instructions for Wasm EH

aheejin updated this revision to Diff 147696. aheejin added a comment. Test case fix was missing in the last commit Repository: rC Clang https://reviews.llvm.org/D44931 Files: lib/CodeGen/CGCXXABI.h lib/CodeGen/CGCleanup.cpp lib/CodeGen/CGCleanup.h lib/CodeGen/CGException.cpp

[PATCH] D44931: [WebAssembly] Use Windows EH instructions for Wasm EH

aheejin updated this revision to Diff 147695. aheejin added a comment. - Make `wasm.get.exception/selector` intrinsics take a token argument Repository: rC Clang https://reviews.llvm.org/D44931 Files: lib/CodeGen/CGCXXABI.h lib/CodeGen/CGCleanup.cpp lib/CodeGen/CGCleanup.h

[PATCH] D44931: [WebAssembly] Use Windows EH instructions for Wasm EH

aheejin added inline comments. Comment at: lib/CodeGen/CGException.cpp:1241-1245 +while (llvm::TerminatorInst *TI = RethrowBlock->getTerminator()) { + llvm::BranchInst *BI = cast(TI); + assert(BI->isConditional()); + RethrowBlock = BI->getSuccessor(1); +}

[PATCH] D44931: [WebAssembly] Use Windows EH instructions for Wasm EH

aheejin updated this revision to Diff 147287. aheejin marked an inline comment as done. aheejin added a comment. `CatchStartBlock` -> `WasmCatchStartBlock` Repository: rC Clang https://reviews.llvm.org/D44931 Files: lib/CodeGen/CGCXXABI.h lib/CodeGen/CGCleanup.cpp

[PATCH] D44931: [WebAssembly] Use Windows EH instructions for Wasm EH

aheejin added a comment. Thank you for the reviews! Comment at: lib/CodeGen/CGException.cpp:1173 +cast(CatchStartBlock->getFirstNonPHI()); +CurrentFuncletPad = CPI; + } majnemer wrote: > Hmm, why is this done? Won't RestoreCurrentFuncletPad undo

[PATCH] D44931: [WebAssembly] Use Windows EH instructions for Wasm EH

aheejin added inline comments. Comment at: lib/CodeGen/CGException.cpp:1241-1245 +while (llvm::TerminatorInst *TI = RethrowBlock->getTerminator()) { + llvm::BranchInst *BI = cast(TI); + assert(BI->isConditional()); + RethrowBlock = BI->getSuccessor(1); +}

[PATCH] D46906: [clangd] Fix a link failure in unittests

aheejin created this revision. aheejin added a reviewer: sammccall. Herald added subscribers: cfe-commits, jkorous, MaskRay, ioeric, ilya-biryukov, mgorny, klimek. aheejin edited the summary of this revision. https://reviews.llvm.org/D46524 (https://reviews.llvm.org/rL332378) introduced a link

[PATCH] D44931: [WebAssembly] Use Windows EH instructions for Wasm EH

aheejin added a comment. Thanks! No, it does not depend on the other CL chain on the llvm side. Repository: rC Clang https://reviews.llvm.org/D44931 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D44931: [WebAssembly] Use Windows EH instructions for Wasm EH

aheejin added a comment. Oh no, sorry, it actually depends on https://reviews.llvm.org/D43746; but not others on the chain. I changed the signatures of `wasm.get.exception` and `wasm.get.ehselector` intrinsics to take a token argument, and the intrinsic signature change is added in

[PATCH] D47732: [clang-doc] Add BitReader to LLVM_LINK_COMPONENTS

aheejin created this revision. aheejin added a reviewer: juliehockett. Herald added subscribers: cfe-commits, mgorny, klimek. Without this, build with `-DSHARED_LIB=ON` fails. Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D47732 Files: clang-doc/CMakeLists.txt Index:

[PATCH] D47620: Remove llvm::Triple argument from get***Personality() functions. NFC.

This revision was automatically updated to reflect the committed changes. Closed by commit rC333938: Remove llvm::Triple argument from get***Personality() functions. NFC. (authored by aheejin, committed by ). Changed prior to commit: https://reviews.llvm.org/D47620?vs=149406=149814#toc

[PATCH] D47732: [clang-doc] Add BitReader to LLVM_LINK_COMPONENTS

aheejin abandoned this revision. aheejin added a comment. Oh I see, thanks! Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D47732 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D44931: [WebAssembly] Use Windows EH instructions for Wasm EH

This revision was automatically updated to reflect the committed changes. Closed by commit rL333703: [WebAssembly] Use Windows EH instructions for Wasm EH (authored by aheejin, committed by ). Herald added a subscriber: llvm-commits. Repository: rL LLVM https://reviews.llvm.org/D44931 Files:

[PATCH] D47614: [WebAssembly] Hide new Wasm EH behind its feature flag

This revision was automatically updated to reflect the committed changes. Closed by commit rC333716: [WebAssembly] Hide new Wasm EH behind its feature flag (authored by aheejin, committed by ). Changed prior to commit: https://reviews.llvm.org/D47614?vs=149386=149387#toc Repository: rC

[PATCH] D47614: [WebAssembly] Hide new Wasm EH behind its feature flag

aheejin created this revision. aheejin added reviewers: dschuff, majnemer. Herald added subscribers: cfe-commits, sunfish, jgravelle-google, sbc100. clang's current wasm EH implementation is a non-MVP feature in progress. We had a `-mexception-handling` wasm feature but were not using it. This

[PATCH] D47614: [WebAssembly] Hide new Wasm EH behind its feature flag

aheejin updated this revision to Diff 149386. aheejin added a comment. - Add a missing conditional statement. I'll still assume this as accepted Repository: rC Clang https://reviews.llvm.org/D47614 Files: lib/CodeGen/CGException.cpp lib/CodeGen/ItaniumCXXABI.cpp

[PATCH] D47614: [WebAssembly] Hide new Wasm EH behind its feature flag

aheejin marked an inline comment as done. aheejin added inline comments. Comment at: lib/CodeGen/CGException.cpp:322 const EHPersonality = - getCXXPersonality(getTarget().getTriple(), LangOpts); + getCXXPersonality(getTarget().getTriple(), LangOpts, getTarget());

[PATCH] D47620: Remove llvm::Triple argument from get***Personality() functions

aheejin created this revision. aheejin added a reviewer: sbc100. Herald added a subscriber: cfe-commits. Because `llvm::Triple` can be derived from `TargetInfo`, it is simpler to take only `TargetInfo` argument. Repository: rC Clang https://reviews.llvm.org/D47620 Files:

[PATCH] D47620: Remove llvm::Triple argument from get***Personality() functions. NFC.

aheejin added a comment. @sbc100 ping :) Repository: rC Clang https://reviews.llvm.org/D47620 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D43437: Fix link failures for Preprocessor::addCommentHandler

aheejin created this revision. aheejin added a reviewer: ioeric. Herald added subscribers: cfe-commits, jkorous-apple, ilya-biryukov, mgorny, klimek. https://reviews.llvm.org/D42640 adds calls to `Preprocessor::addCommentHandler` in `unittests/clangd/SymbolCollectorTests.cpp` and

[PATCH] D43681: [WebAssembly] Add exception handling option

aheejin created this revision. aheejin added a reviewer: dschuff. Herald added subscribers: cfe-commits, sunfish, jgravelle-google, sbc100, jfb. Add exception handling option to clang. Repository: rC Clang https://reviews.llvm.org/D43681 Files: docs/ClangCommandLineReference.rst

[PATCH] D43437: [clangd] Fix link failures for Preprocessor::addCommentHandler

This revision was automatically updated to reflect the committed changes. Closed by commit rL325458: [clangd] Fix link failures for Preprocessor::addCommentHandler (authored by aheejin, committed by ). Herald added a subscriber: llvm-commits. Repository: rL LLVM

[PATCH] D48443: [WIP] Add no-prototype attribute to prototype-less C functions

aheejin added inline comments. Comment at: lib/CodeGen/CGCall.cpp:1849 + if (!AttrOnCallSite && !Fn->hasPrototype()) +FuncAttrs.addAttribute("no-prototype"); AddAttributesFromFunctionProtoType( Is there a reason why this is not something like

[PATCH] D49396: [WebAssembly] Support for atomic.wait / atomic.wake builtins

aheejin updated this revision to Diff 158555. aheejin added a comment. - wake -> notify Repository: rC Clang https://reviews.llvm.org/D49396 Files: include/clang/Basic/BuiltinsWebAssembly.def lib/CodeGen/CGBuiltin.cpp test/CodeGen/builtins-wasm.c Index: test/CodeGen/builtins-wasm.c

[PATCH] D49396: [WebAssembly] Support for atomic.wait / atomic.wake builtins

This revision was automatically updated to reflect the committed changes. Closed by commit rL338771: [WebAssembly] Support for atomic.wait / atomic.wake builtins (authored by aheejin, committed by ). Herald added a subscriber: llvm-commits. Repository: rL LLVM https://reviews.llvm.org/D49396

[PATCH] D49396: [WebAssembly] Support for atomic.wait / atomic.wake builtins

aheejin updated this revision to Diff 158017. aheejin added a comment. Herald added a subscriber: jfb. - Changed the first argument of wake from i8* to i32* Repository: rC Clang https://reviews.llvm.org/D49396 Files: include/clang/Basic/BuiltinsWebAssembly.def lib/CodeGen/CGBuiltin.cpp

[PATCH] D49396: [WebAssembly] Support for atomic.wait / atomic.wake builtins

aheejin updated this revision to Diff 158434. aheejin marked an inline comment as done. aheejin added a comment. - Changed types Repository: rC Clang https://reviews.llvm.org/D49396 Files: include/clang/Basic/BuiltinsWebAssembly.def lib/CodeGen/CGBuiltin.cpp

[PATCH] D49396: [WebAssembly] Support for atomic.wait / atomic.wake builtins

aheejin added inline comments. Comment at: include/clang/Basic/BuiltinsWebAssembly.def:38 +// Atomic wait and wake. +BUILTIN(__builtin_wasm_atomic_wait_i32, "Uii*iLLi", "n") +BUILTIN(__builtin_wasm_atomic_wait_i64, "UiLLi*LLiLLi", "n") dschuff wrote: > So this

[PATCH] D51013: [WebAssembly] Revert type of wake count in atomic.wake to i32

This revision was automatically updated to reflect the committed changes. Closed by commit rC340235: [WebAssembly] Revert type of wake count in atomic.wake to i32 (authored by aheejin, committed by ). Changed prior to commit: https://reviews.llvm.org/D51013?vs=161615=161620#toc Repository:

[PATCH] D51013: [WebAssembly] Revert type of wake count in atomic.wake to i32

aheejin created this revision. aheejin added a reviewer: dschuff. Herald added subscribers: cfe-commits, jfb, sunfish, jgravelle-google, sbc100. We decided to revert this from i64 to i32 in Nov 28 CG meeting. Fixes PR38632. Repository: rC Clang https://reviews.llvm.org/D51013 Files:

[PATCH] D49396: [WebAssembly] Support for atomic.wait / atomic.wake builtins

aheejin updated this revision to Diff 155958. aheejin added a comment. - Type fix Repository: rC Clang https://reviews.llvm.org/D49396 Files: include/clang/Basic/BuiltinsWebAssembly.def lib/CodeGen/CGBuiltin.cpp test/CodeGen/builtins-wasm.c Index: test/CodeGen/builtins-wasm.c

[PATCH] D49396: [WebAssembly] Support for atomic.wait / atomic.wake builtins

aheejin created this revision. aheejin added a reviewer: dschuff. Herald added subscribers: cfe-commits, sunfish, jgravelle-google, sbc100. Add support for atomic.wait / atomic.wake builtins based on the Wasm thread proposal. Repository: rC Clang https://reviews.llvm.org/D49396 Files:

[PATCH] D49396: [WebAssembly] Support for atomic.wait / atomic.wake builtins

aheejin updated this revision to Diff 155738. aheejin added a comment. - test fix Repository: rC Clang https://reviews.llvm.org/D49396 Files: include/clang/Basic/BuiltinsWebAssembly.def lib/CodeGen/CGBuiltin.cpp test/CodeGen/builtins-wasm.c Index: test/CodeGen/builtins-wasm.c

[PATCH] D49629: [CMake] Fix the setting of LIBCXX_HEADER_DIR

aheejin added a comment. Ping Repository: rCXX libc++ https://reviews.llvm.org/D49629 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D49629: [CMake] Fix the setting of LIBCXX_HEADER_DIR

This revision was automatically updated to reflect the committed changes. Closed by commit rL337727: [CMake] Fix the setting of LIBCXX_HEADER_DIR (authored by aheejin, committed by ). Herald added a subscriber: llvm-commits. Repository: rL LLVM https://reviews.llvm.org/D49629 Files:

[PATCH] D49711: [CMake] Fix the setting of LIBCXX_HEADER_DIR in standalone build

aheejin added a comment. This fixes our local waterfall build failure. Thank you! Repository: rCXX libc++ https://reviews.llvm.org/D49711 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D49629: [CMake] Fix the setting of LIBCXX_HEADER_DIR

aheejin created this revision. aheejin added a reviewer: phosek. Herald added subscribers: cfe-commits, ldionne, christof, mgorny. Repository: rCXX libc++ https://reviews.llvm.org/D49629 Files: CMakeLists.txt Index: CMakeLists.txt

[PATCH] D49629: [CMake] Fix the setting of LIBCXX_HEADER_DIR

aheejin added a comment. This line was added in https://reviews.llvm.org/rL335809, and after https://reviews.llvm.org/rL337630 `dst` in this line is set to NULL in our

[PATCH] D51448: [WebAssembly] clang-format (NFC)

aheejin added a comment. Ping! Can we land this because this is fairly small? Repository: rC Clang https://reviews.llvm.org/D51448 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D51448: [WebAssembly] clang-format (NFC)

This revision was automatically updated to reflect the committed changes. Closed by commit rL341254: [WebAssembly] clang-format (NFC) (authored by aheejin, committed by ). Herald added a subscriber: llvm-commits. Repository: rL LLVM https://reviews.llvm.org/D51448 Files:

[PATCH] D51448: [WebAssembly] clang-format (NFC)

This revision was automatically updated to reflect the committed changes. Closed by commit rC341254: [WebAssembly] clang-format (NFC) (authored by aheejin, committed by ). Changed prior to commit: https://reviews.llvm.org/D51448?vs=163152=163590#toc Repository: rL LLVM

[PATCH] D51448: [WebAssembly] clang-format (NFC)

aheejin added a comment. Ping pong Repository: rC Clang https://reviews.llvm.org/D51448 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D51448: [WebAssembly] clang-format (NFC)

aheejin created this revision. Herald added subscribers: cfe-commits, jfb, sunfish, jgravelle-google, sbc100, dschuff. This patch runs clang-format on all wasm-only files. Repository: rC Clang https://reviews.llvm.org/D51448 Files: lib/Driver/ToolChains/WebAssembly.cpp

[PATCH] D44931: [WebAssembly] Use Windows EH instructions for Wasm EH

aheejin updated this revision to Diff 141979. aheejin added a comment. - Change personality function name to a unique one Repository: rC Clang https://reviews.llvm.org/D44931 Files: lib/CodeGen/CGCXXABI.h lib/CodeGen/CGCleanup.cpp lib/CodeGen/CGCleanup.h lib/CodeGen/CGException.cpp

[PATCH] D44931: [WebAssembly] Use Windows EH instructions for Wasm EH

aheejin updated this revision to Diff 141983. aheejin added a comment. - Rebase Repository: rC Clang https://reviews.llvm.org/D44931 Files: lib/CodeGen/CGCXXABI.h lib/CodeGen/CGCleanup.cpp lib/CodeGen/CGCleanup.h lib/CodeGen/CGException.cpp lib/CodeGen/ItaniumCXXABI.cpp

[PATCH] D45555: Fix a link failure in applyChanges()

aheejin created this revision. aheejin added a reviewer: jdemeule. Herald added subscribers: cfe-commits, mgorny, klimek. This call (https://github.com/llvm-mirror/clang-tools-extra/blob/e6bfa666d96c0d3010bb7d572f6240424ebd1cff/clang-apply-replacements/lib/Tooling/ApplyReplacements.cpp#L228-L229)

[PATCH] D45555: [clang-apply-replacements] Fix a link failure in applyChanges()

aheejin abandoned this revision. aheejin added a comment. Looks like the same patch https://reviews.llvm.org/rL329892 has already landed. Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D4 ___ cfe-commits mailing list

[PATCH] D44931: [WebAssembly] Use Windows EH instructions for Wasm EH

aheejin updated this revision to Diff 141759. aheejin added a comment. - Add a test in which try-catches are nested within a catch - Rebase Repository: rC Clang https://reviews.llvm.org/D44931 Files: lib/CodeGen/CGCXXABI.h lib/CodeGen/CGCleanup.cpp lib/CodeGen/CGCleanup.h

[PATCH] D44931: [WebAssembly] Use Windows EH instructions for Wasm EH

aheejin created this revision. aheejin added reviewers: majnemer, dschuff. Herald added subscribers: cfe-commits, sunfish, jgravelle-google, sbc100, jfb. Because wasm control flow needs to be structured, using WinEH instructions to support wasm EH brings several benefits. This patch makes wasm EH

[PATCH] D44931: [WebAssembly] Use Windows EH instructions for Wasm EH

aheejin updated this revision to Diff 139920. aheejin added a comment. - Comment fix Repository: rC Clang https://reviews.llvm.org/D44931 Files: lib/CodeGen/CGCXXABI.h lib/CodeGen/CGCleanup.cpp lib/CodeGen/CGCleanup.h lib/CodeGen/CGException.cpp lib/CodeGen/ItaniumCXXABI.cpp

[PATCH] D44931: [WebAssembly] Use Windows EH instructions for Wasm EH

aheejin updated this revision to Diff 140144. aheejin marked an inline comment as done. aheejin added a comment. - Rebase & Simplified the if condition Repository: rC Clang https://reviews.llvm.org/D44931 Files: lib/CodeGen/CGCXXABI.h lib/CodeGen/CGCleanup.cpp lib/CodeGen/CGCleanup.h

[PATCH] D44931: [WebAssembly] Use Windows EH instructions for Wasm EH

aheejin added inline comments. Comment at: lib/CodeGen/CGException.cpp:1541 + } llvm::CallInst *terminateCall = + CGM.getCXXABI().emitTerminateForUnexpectedException(*this, Exn); dschuff wrote: > Should this be in an else block? No need to emit it after

[PATCH] D44931: [WebAssembly] Use Windows EH instructions for Wasm EH

aheejin added inline comments. Comment at: lib/CodeGen/CGCXXABI.h:610 +struct CatchRetScope final : EHScopeStack::Cleanup { + llvm::CatchPadInst *CPI; dschuff wrote: > Should be `public`? This code was moved from [[

[PATCH] D44931: [WebAssembly] Use Windows EH instructions for Wasm EH

aheejin updated this revision to Diff 140156. aheejin marked an inline comment as done. aheejin added a comment. - GNU_CPlusCPlus -> GNU_CPlusPlus Repository: rC Clang https://reviews.llvm.org/D44931 Files: lib/CodeGen/CGCXXABI.h lib/CodeGen/CGCleanup.cpp lib/CodeGen/CGCleanup.h

[PATCH] D43681: [WebAssembly] Add exception handling option

This revision was automatically updated to reflect the committed changes. Closed by commit rL326517: [WebAssembly] Add exception handling option (authored by aheejin, committed by ). Herald added a subscriber: llvm-commits. Repository: rL LLVM https://reviews.llvm.org/D43681 Files:

[PATCH] D53685: [WebAssembly] Bitselect and min/max builtins

aheejin added inline comments. Comment at: include/clang/Basic/BuiltinsWebAssembly.def:85 +BUILTIN(__builtin_wasm_bitselect, "V4iV4iV4iV4i", "nc") + Do you think we need to provide this builtin also for other vector types? Repository: rC Clang

[PATCH] D53724: [WebAssembly] Fix type names in truncation builtins

aheejin added inline comments. Comment at: lib/CodeGen/CGBuiltin.cpp:12528 + case WebAssembly::BI__builtin_wasm_trunc_saturate_s_f32x4: + case WebAssembly::BI__builtin_wasm_trunc_saturate_s_f64x2: { Value *Src = EmitScalarExpr(E->getArg(0)); tlively

[PATCH] D53675: [WebAssembly] Lower select for vectors

aheejin added a comment. The commit here looks like replaced with that of https://reviews.llvm.org/D53724. Repository: rC Clang https://reviews.llvm.org/D53675 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D53724: [WebAssembly] Fix type names in truncation builtins

aheejin added inline comments. Comment at: lib/CodeGen/CGBuiltin.cpp:12528 + case WebAssembly::BI__builtin_wasm_trunc_saturate_s_f32x4: + case WebAssembly::BI__builtin_wasm_trunc_saturate_s_f64x2: { Value *Src = EmitScalarExpr(E->getArg(0)); But unlike

[PATCH] D53675: [WebAssembly] Lower select for vectors

aheejin added a comment. I tried to add `ISD::SELECT` in the expansion list, like for (auto Op : {ISD::VSELECT, ISD::SELECT_CC, ISD::SELECT}) { ... } And the code generation apparently works for current test `simd-select.ll`. I'm not sure if it's optimal though. Could you check again?

[PATCH] D54572: [WebAssembly] Change type of wake count to unsigned int

This revision was automatically updated to reflect the committed changes. Closed by commit rL347005: [WebAssembly] Change type of wake count to unsigned int (authored by aheejin, committed by ). Herald added a subscriber: llvm-commits. Repository: rL LLVM https://reviews.llvm.org/D54572

[PATCH] D54572: [WebAssembly] Change type of wake count to unsigned int

This revision was automatically updated to reflect the committed changes. Closed by commit rC347005: [WebAssembly] Change type of wake count to unsigned int (authored by aheejin, committed by ). Changed prior to commit: https://reviews.llvm.org/D54572?vs=174169=174302#toc Repository: rL

[PATCH] D54572: [WebAssembly] Change type of wake count to unsigned int

aheejin created this revision. aheejin added a reviewer: dschuff. Herald added subscribers: cfe-commits, jfb, sunfish, jgravelle-google, sbc100. We discussed this at the Nov 12th CG meeting, and decided to use the unsigned semantics for the wake count. Corresponding spec change:

[PATCH] D53007: [WebAssembly][NFC] Rename test functions for builtins

aheejin accepted this revision. aheejin added a comment. This revision is now accepted and ready to land. A lot better this way! Thanks. Repository: rC Clang https://reviews.llvm.org/D53007 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D52852: [WebAssembly] __builtin_wasm_extract_lane_* builtins

aheejin added inline comments. Comment at: include/clang/Basic/BuiltinsWebAssembly.def:46 +BUILTIN(__builtin_wasm_extract_lane_s_i16x8, "iV8sIi", "ncV:128:") +BUILTIN(__builtin_wasm_extract_lane_u_i16x8, "iV8sIi", "ncV:128:") +BUILTIN(__builtin_wasm_extract_lane_i32x4, "iV4iIi",

[PATCH] D52856: [WebAssembly] __builtin_wasm_replace_lane_* builtins

aheejin added inline comments. Comment at: include/clang/Basic/BuiltinsWebAssembly.def:55 +BUILTIN(__builtin_wasm_replace_lane_i32x4, "V4iV4iIii", "ncV:128:") +BUILTIN(__builtin_wasm_replace_lane_i64x2, "V2WiV2WiIiWi", "ncV:128:") +BUILTIN(__builtin_wasm_replace_lane_f32x4,

[PATCH] D56504: [WebAssembly] Add unimplemented-simd128 feature, gate builtins

aheejin added inline comments. Comment at: include/clang/Basic/BuiltinsWebAssembly.def:53 // Saturating fp-to-int conversions -BUILTIN(__builtin_wasm_trunc_saturate_s_i32_f32, "if", "nc") -BUILTIN(__builtin_wasm_trunc_saturate_u_i32_f32, "if", "nc")

[PATCH] D56504: [WebAssembly] Add simd128-unimplemented feature, gate builtins

aheejin added inline comments. Comment at: include/clang/Basic/BuiltinsWebAssembly.def:53 // Saturating fp-to-int conversions -BUILTIN(__builtin_wasm_trunc_saturate_s_i32_f32, "if", "nc") -BUILTIN(__builtin_wasm_trunc_saturate_u_i32_f32, "if", "nc")

[PATCH] D59827: [slh] x86 impl of ARM instrinsic for SLH

aheejin added inline comments. Comment at: clang/test/Preprocessor/init.c:9678 // WEBASSEMBLY-NEXT:#define __GXX_ABI_VERSION 1002 +// WEBASSEMBLY-NEXT: #define __HAVE_SPECULATION_SAFE_VALUE 1 // WEBASSEMBLY32-NEXT:#define __ILP32__ 1 Nit: Remove the whitespace

[PATCH] D58742: [WebAssembly] Remove uses of ThreadModel

aheejin accepted this revision. aheejin added a comment. This looks a nice improvement! Thank you. Comment at: clang/include/clang/Driver/ToolChain.h:456 /// getThreadModel() - Which thread model does this target use? - virtual std::string getThreadModel(const

[PATCH] D58742: [WebAssembly] Remove uses of ThreadModel

aheejin added inline comments. Comment at: clang/include/clang/Driver/ToolChain.h:456 /// getThreadModel() - Which thread model does this target use? - virtual std::string getThreadModel(const llvm::opt::ArgList &) const { -return "posix"; - } + virtual std::string

[PATCH] D59353: [WebAssembly] Use rethrow intrinsic in the rethrow block

aheejin created this revision. aheejin added a reviewer: dschuff. Herald added subscribers: cfe-commits, sunfish, jgravelle-google, sbc100. Herald added a project: clang. Because in wasm we merge all catch clauses into one big catchpad, in case none of the types in catch handlers matches after we

[PATCH] D59448: [WebAssembly] Change wasm.throw's first argument to an immediate

aheejin added a comment. In D59448#1433607 , @dschuff wrote: > LGTM; I wonder if it makes sense to have predefined macro for the C++ tag (or > perhaps just a regular macro for use in libcxxabi?) The only place the `throw` builtin will be used in within

[PATCH] D59448: [WebAssembly] Change wasm.throw's first argument to an immediate

This revision was automatically updated to reflect the committed changes. Closed by commit rC356436: [WebAssembly] Change wasm.throws first argument to an immediate (authored by aheejin, committed by ). Changed prior to commit: https://reviews.llvm.org/D59448?vs=191240=191241#toc Repository:

[PATCH] D59448: [WebAssembly] Change wasm.throw's first argument to an immediate

aheejin added a comment. In D59448#1433622 , @craig.topper wrote: > Shouldn't the definition in BuiltinsWebAssembly.def be updated to include an > 'I' in the type string so that this will be properly diagnosed in the > frontend? Done. Thank you for

[PATCH] D59448: [WebAssembly] Change wasm.throw's first argument to an immediate

aheejin updated this revision to Diff 191240. aheejin added a comment. - Add `I` to `wasm.throw` builtin's tag argument Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D59448/new/ https://reviews.llvm.org/D59448 Files: include/clang/Basic/BuiltinsWebAssembly.def

[PATCH] D59353: [WebAssembly] Use rethrow intrinsic in the rethrow block

This revision was automatically updated to reflect the committed changes. Closed by commit rC356317: [WebAssembly] Use rethrow intrinsic in the rethrow block (authored by aheejin, committed by ). Changed prior to commit: https://reviews.llvm.org/D59353?vs=190587=190947#toc Repository: rC

[PATCH] D59448: [WebAssembly] Change wasm.throw intrinsic's first argument to immediate

aheejin created this revision. aheejin added a reviewer: dschuff. Herald added subscribers: cfe-commits, sunfish, jgravelle-google, sbc100. Herald added a project: clang. After D57825 (r355981), intrinsic arguments that are marked as `ImmArg` only can take

[PATCH] D57874: [WebAssembly] Set '-matomics' when '-pthread' is set

aheejin updated this revision to Diff 185895. aheejin added a comment. Sorry nevermind my previous code. There was not hacky and much cleaner way to do everything in the driver layer. (Before I tried to do everything in the cc1 compilation layer :( ) Anyway, moved all logic to the driver layer

[PATCH] D57874: [WebAssembly] Set '-matomics' when '-pthread' is set

aheejin marked an inline comment as done. aheejin added inline comments. Comment at: lib/Driver/ToolChains/WebAssembly.cpp:66 +if (Args.hasFlag(clang::driver::options::OPT_pthread, + clang::driver::options::OPT_no_pthread), +false)

[PATCH] D57874: [WebAssembly] Make thread-related options consistent

aheejin marked an inline comment as done. aheejin added inline comments. Comment at: lib/Driver/ToolChains/WebAssembly.cpp:66 +if (Args.hasFlag(clang::driver::options::OPT_pthread, + clang::driver::options::OPT_no_pthread), +false)

[PATCH] D57874: [WebAssembly] Make thread-related options consistent

aheejin updated this revision to Diff 186065. aheejin added a comment. - Small fix Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57874/new/ https://reviews.llvm.org/D57874 Files: include/clang/Driver/ToolChain.h lib/Driver/Driver.cpp

[PATCH] D57874: [WebAssembly] Make thread-related options consistent

aheejin added a comment. In D57874#1389981 , @sunfish wrote: > > - `-matomics` means `-mthread-model posix` > > The others sound reasonable, though this one seems a little surprising -- a > user might have -matomics enabled because they're targeting a VM

[PATCH] D57874: [WebAssembly] Make thread-related options consistent

aheejin added a comment. In D57874#1391133 , @tlively wrote: > In D57874#1389953 , @aheejin wrote: > > > Anyway, moved all logic to the driver layer and did this: > > > > - `-matomics` means `-mthread-model posix`

[PATCH] D57874: [WebAssembly] Make thread-related options consistent

aheejin updated this revision to Diff 186063. aheejin added a comment. - Fix some bugs - Make the driver error out when explicitly given options don't match, such as `-no-pthread` and `-matomics` are given at the same time Repository: rC Clang CHANGES SINCE LAST ACTION

[PATCH] D57874: [WebAssembly] Make thread-related options consistent

aheejin updated this revision to Diff 186068. aheejin added a comment. - Initialized ThreadModel vairables with "" Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57874/new/ https://reviews.llvm.org/D57874 Files: include/clang/Driver/ToolChain.h

[PATCH] D57874: [WebAssembly] Make thread-related options consistent

aheejin added a comment. I only added those test routines in `wasm-toolchain.c` and not `wasm-toolchain.cpp`, because they are basically the same. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57874/new/ https://reviews.llvm.org/D57874

[PATCH] D57736: [WebAssembly] Bulk memory intrinsics and builtins

aheejin accepted this revision. aheejin added inline comments. This revision is now accepted and ready to land. Comment at: clang/lib/CodeGen/CGBuiltin.cpp:13477 +if (!SegArg->isIntegerConstantExpr(SegConst, getContext())) + llvm_unreachable("Constant arg isn't actually

[PATCH] D57736: [WebAssembly] Bulk memory intrinsics and builtins

aheejin added inline comments. Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Comment at: clang/include/clang/Basic/BuiltinsWebAssembly.def:29 +// Bulk memory builtins +TARGET_BUILTIN(__builtin_wasm_memory_init, "vIiv*ii", "", "bulk-memory")

[PATCH] D57874: [WebAssembly] Make thread-related options consistent

aheejin added a comment. If people have opinions on this final version, please let me know. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57874/new/ https://reviews.llvm.org/D57874 ___ cfe-commits mailing list

[PATCH] D57874: [WebAssembly] Make thread-related options consistent

This revision was automatically updated to reflect the committed changes. Closed by commit rL353761: [WebAssembly] Make thread-related options consistent (authored by aheejin, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Repository: rL LLVM CHANGES

[PATCH] D57874: [WebAssembly] Set '-matomics' when '-pthread' is set

aheejin created this revision. aheejin added a reviewer: dschuff. Herald added subscribers: cfe-commits, jfb, sunfish, jgravelle-google, sbc100. Herald added a project: clang. In wasm, we always use '-matomics' when we use '-pthread'. This will make users type one less option and options will be

[PATCH] D57874: [WebAssembly] Set '-matomics' when '-pthread' is set

aheejin updated this revision to Diff 185700. aheejin added a comment. - Small fix Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57874/new/ https://reviews.llvm.org/D57874 Files: lib/Frontend/CompilerInvocation.cpp test/Driver/wasm32-unknown-unknown.cpp

[PATCH] D57798: [WebAssembly] Add atomics target option

aheejin created this revision. aheejin added a reviewer: tlively. Herald added subscribers: cfe-commits, jfb, sunfish, jgravelle-google, sbc100, dschuff. Herald added a project: clang. Repository: rC Clang https://reviews.llvm.org/D57798 Files: include/clang/Driver/Options.td

  1   2   3   >