[PATCH] D156089: [Driver][XRay][test] Convert more tests to check 'target=...' after 016785d9316d8c5abc5fdf3cdb86479095bbb677

SixWeining created this revision. SixWeining added reviewers: probinson, MaskRay. Herald added a subscriber: dberris. Herald added a project: All. SixWeining requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Fix https://github.com/llvm/llvm-p

[PATCH] D156089: [Driver][XRay][test] Convert more tests to check 'target=...' after 016785d9316d8c5abc5fdf3cdb86479095bbb677

SixWeining added inline comments. Comment at: clang/test/Driver/XRay/xray-ignore-loops-flags.cpp:5 // // RUN: %clang -fxray-instrument -fxray-ignore-loops -target x86_64-linux- -### \ // RUN: -x c++ -emit-llvm -c -o - %s 2>&1 | FileCheck %s `x86_64-linux` i

[PATCH] D156114: [Clang][LoongArch] Pre-commit test for Dxxxxxx

SixWeining created this revision. SixWeining added reviewers: xry111, xen0n, hev, wangleiat. Herald added a project: All. SixWeining requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github Monorepo https://reviews.llvm

[PATCH] D156116: [Clang][LoongArch] Fix ABI handling of empty structs in C++ to match GCC behaviour

SixWeining created this revision. SixWeining added reviewers: xry111, xen0n, hev, wangleiat. Herald added subscribers: luismarques, s.egerton, PkmX, simoncook, arichardson. Herald added a project: All. SixWeining requested review of this revision. Herald added subscribers: cfe-commits, wangpc. Hera

[PATCH] D156114: [Clang][LoongArch] Pre-commit test for D156116

This revision was not accepted when it landed; it landed in state "Needs Review". This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG669d2bbc837e: [Clang][LoongArch] Pre-commit test for D156116 (auth

[PATCH] D156116: [Clang][LoongArch] Fix ABI handling of empty structs in C++ to match GCC behaviour

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGc299efbf284f: [Clang][LoongArch] Fix ABI handling of empty structs in C++ to match GCC… (authored by SixWeining). Repository: rG LLVM Github Monor

[PATCH] D154931: [LoongArch] Support InlineAsm for LSX and LASX

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGd25c79dc7000: [LoongArch] Support InlineAsm for LSX and LASX (authored by chenli, committed by SixWeining). Repository: rG LLVM Github Monorepo C

[PATCH] D156116: [Clang][LoongArch] Fix ABI handling of empty structs in C++ to match GCC behaviour

SixWeining added a comment. In D156116#4530365 , @thakis wrote: > This seems to not build: http://45.33.8.238/linux/113405/step_4.txt (?) Thanks, I have reverted. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.o

[PATCH] D156089: [Driver][XRay][test] Convert more tests to check 'target=...' after 016785d9316d8c5abc5fdf3cdb86479095bbb677

SixWeining abandoned this revision. SixWeining added a comment. 841ff7c6ca4c6fa9ff3ac6ba93e177462b7d668e addresses this issue. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D1560

[PATCH] D155824: [LoongArch] Support -march=native and -mtune=

SixWeining added a comment. @xen0n Do you have any inputs? I think supporting these options can improve compatibility with gcc (although there is no ScheduleModel difference among currently supported processors) and I hope it can be merged into LLVM17. Repository: rG LLVM Github Monorepo CH

[PATCH] D156195: [docs] Add llvm & clang release notes for LoongArch

SixWeining created this revision. SixWeining added reviewers: xen0n, xry111, hev, wangleiat. Herald added a project: All. SixWeining requested review of this revision. Herald added projects: clang, LLVM. Herald added subscribers: llvm-commits, cfe-commits. Repository: rG LLVM Github Monorepo ht

[PATCH] D142327: [clang][RISCV] Fix ABI handling of empty structs with hard FP calling conventions in C++

SixWeining added inline comments. Comment at: clang/lib/CodeGen/Targets/RISCV.cpp:178 return false; -if (isEmptyRecord(getContext(), Ty, true)) +if (isEmptyRecord(getContext(), Ty, true, true)) return true; asb wrote: > rogfer01 wrote: > > I

[PATCH] D156195: [docs] Add llvm & clang release notes for LoongArch

SixWeining updated this revision to Diff 543867. SixWeining added a comment. Herald added a reviewer: alexander-shaposhnikov. Address @xen0n's comments and add some more recordable updates. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D156195/new/

[PATCH] D156195: [docs] Add llvm & clang release notes for LoongArch

SixWeining updated this revision to Diff 543876. SixWeining added a comment. Move sanitizer changes to clang side. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D156195/new/ https://reviews.llvm.org/D156195 Files: clang/docs/ReleaseNotes.rst ll

[PATCH] D156195: [docs] Add llvm & clang release notes for LoongArch

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG6223050db88b: [docs] Add llvm & clang release notes for LoongArch (authored by SixWeining). Repository: rG LLVM Github Monorepo CHANGES SINCE LAS

[PATCH] D155824: [LoongArch] Support -march=native and -mtune=

SixWeining updated this revision to Diff 543902. SixWeining added a comment. update clang release notes Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D155824/new/ https://reviews.llvm.org/D155824 Files: clang/docs/ReleaseNotes.rst clang/lib/Bas

[PATCH] D155824: [LoongArch] Support -march=native and -mtune=

SixWeining updated this revision to Diff 543919. SixWeining added a comment. Address @xen0n's comments. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D155824/new/ https://reviews.llvm.org/D155824 Files: clang/docs/ReleaseNotes.rst clang/lib/Bas

[PATCH] D155824: [LoongArch] Support -march=native and -mtune=

SixWeining updated this revision to Diff 543941. SixWeining added a comment. add missing `.` Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D155824/new/ https://reviews.llvm.org/D155824 Files: clang/docs/ReleaseNotes.rst clang/lib/Basic/Targets/

[PATCH] D155824: [LoongArch] Support -march=native and -mtune=

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG92c06114b2ea: [LoongArch] Support -march=native and -mtune= (authored by SixWeining). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTI

[PATCH] D155824: [LoongArch] Support -march=native and -mtune=

SixWeining added inline comments. Comment at: clang/docs/ReleaseNotes.rst:138-143 +- Patchable function entry (``-fpatchable-function-entry``) is now supported + on LoongArch. +- Unaligned memory accesses can be toggled by ``-m[no-]unaligned-access`` or the + aliases ``-m[no-]

[PATCH] D155824: [LoongArch] Support -march=native and -mtune=

SixWeining added a comment. In D155824#4548677 , @steven_wu wrote: > After another look, I will need to request to revert this because this > implementation doesn't work with `-fno-integrated-cc1` at all. You can't > setCPU/Tune in the driver into a glo

[PATCH] D155824: [LoongArch] Support -march=native and -mtune=

SixWeining updated this revision to Diff 546730. SixWeining edited the summary of this revision. SixWeining added a comment. - work with `-fno-integrated-cc1` - add mingsing `"`s in macros Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D155824/new/ h

[PATCH] D155824: [LoongArch] Support -march=native and -mtune=

SixWeining updated this revision to Diff 546742. SixWeining marked an inline comment as done. SixWeining added a comment. Add a common helper for `-m{arch,tune}=native` handling. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D155824/new/ https://rev

[PATCH] D155824: [LoongArch] Support -march=native and -mtune=

SixWeining added inline comments. Comment at: clang/lib/Driver/ToolChains/Arch/LoongArch.cpp:199-204 + // Handle CPU name is 'native'. + if (CPU == "native") { +CPU = llvm::sys::getHostCPUName(); +if (CPU == "generic") + CPU = llvm::LoongArch::getDefaultArch(Triple

[PATCH] D155824: [LoongArch] Support -march=native and -mtune=

SixWeining updated this revision to Diff 546766. SixWeining added a comment. rename the common helper to `postProcessTargetCPUString` Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D155824/new/ https://reviews.llvm.org/D155824 Files: clang/lib/Bas

[PATCH] D156866: [Clang][LoongArch] Use the ClangBuiltin class to automatically generate support for CBE and CFE

SixWeining added a comment. LGTM except a nit. Comment at: clang/lib/CodeGen/CGBuiltin.cpp:20447 - -Value *CodeGenFunction::EmitLoongArchBuiltinExpr(unsigned BuiltinID, - const CallExpr *E) { The declaration in `c

[PATCH] D156866: [Clang][LoongArch] Use the ClangBuiltin class to automatically generate support for CBE and CFE

SixWeining accepted this revision. SixWeining added a comment. This revision is now accepted and ready to land. The change is loongarch-specific and it looks goold to me. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D156866/new/ https://reviews.llv

[PATCH] D156116: [Clang][LoongArch] Fix ABI handling of empty structs in C++ to match GCC behaviour

SixWeining updated this revision to Diff 547711. SixWeining added a comment. add an extra check for when a candidate from detectFAREligibleStructHelper may not be valid Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D156116/new/ https://reviews.llvm

[PATCH] D156116: [Clang][LoongArch] Fix ABI handling of empty structs in C++ to match GCC behaviour

This revision was automatically updated to reflect the committed changes. Closed by commit rGe7a8a7d49776: [Clang][LoongArch] Fix ABI handling of empty structs in C++ to match GCC… (authored by SixWeining). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.or

[PATCH] D155824: [LoongArch] Support -march=native and -mtune=

SixWeining updated this revision to Diff 548442. SixWeining marked an inline comment as done. SixWeining added a comment. Address @MaskRay's comments. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D155824/new/ https://reviews.llvm.org/D155824 Files

[PATCH] D155824: [LoongArch] Support -march=native and -mtune=

SixWeining marked an inline comment as done. SixWeining added inline comments. Comment at: clang/lib/Basic/Targets/LoongArch.cpp:203 + StringRef ArchName = getCPU(); + Builder.defineMacro("__loongarch_arch", Twine("\"") + ArchName + Twine("\"")); + MaskRay wro

[PATCH] D155824: [LoongArch] Support -march=native and -mtune=

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. SixWeining marked an inline comment as done. Closed by commit rGf62c9252fc0f: [LoongArch] Support -march=native and -mtune= (authored by SixWeining). Repository: rG

[PATCH] D140727: [XRay] Add initial support for loongarch64

SixWeining added a comment. Since `R_LARCH_64_PCREL` has been supported, we can use version 2 xray and the patch summary should be modified accordingly. Comment at: llvm/lib/Target/LoongArch/LoongArchAsmPrinter.cpp:187 + OutStreamer->emitLabel(EndOfSled); + recordSled(BeginO

[PATCH] D155163: [Triple] Add llvm::Triple::isLoongArch{32,64}

SixWeining updated this revision to Diff 539915. SixWeining added a comment. Herald added subscribers: cfe-commits, Enna1, hiraditya. Herald added a project: clang. Use the new added `isLoongArch64()` in some places. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://revi

[PATCH] D155163: [Triple] Add llvm::Triple::isLoongArch{32,64}

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGa926a2660a7f: [Triple] Add llvm::Triple::isLoongArch{32,64} (authored by SixWeining). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTI

[PATCH] D138350: [scudo] Add loongarch64 support for scudo

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

[PATCH] D138403: [LoongArch] Fix issue on CMake Xcode build configuration

SixWeining accepted this revision. SixWeining added a comment. This revision is now accepted and ready to land. LGTM but wait for @WowbaggersLiquidLunch. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D138403/new/ https://reviews.llvm.org/D138403 __

[PATCH] D138489: [tsan] Add tsan support for loongarch64

SixWeining added a comment. I haven't check the correctness of `tsan_rtl_loongarch64.S`. Comment at: compiler-rt/lib/sanitizer_common/sanitizer_linux.cpp:1513 + + register int res __asm__("$a0"); + register int __flags __asm__("$a0") = flags; I'm not sure if

[PATCH] D138418: [LoongArch] Add remaining intrinsics for CRC check instructions

SixWeining added inline comments. Comment at: llvm/lib/Target/LoongArch/LoongArchISelLowering.cpp:925-933 +case Intrinsic::loongarch_crc_w_b_w: { + Results.push_back(DAG.getNode( + ISD::TRUNCATE, DL, VT, + DAG.getNode(LoongArchISD::CRC_W_B_W, DL, MVT::i

[PATCH] D138350: [scudo] Add loongarch64 support for scudo

This revision was automatically updated to reflect the committed changes. Closed by commit rGac84798570cd: [scudo] Add loongarch64 support for scudo (authored by tangyouling, committed by SixWeining). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D138

[PATCH] D138418: [LoongArch] Add remaining intrinsics for CRC check instructions

SixWeining accepted this revision. SixWeining added a comment. This revision is now accepted and ready to land. LGTM. Comment at: llvm/lib/Target/LoongArch/LoongArchISelLowering.cpp:925 llvm_unreachable("Unexpected Intrinsic."); -case Intrinsic::loongarch_crc_w_d_w:

[PATCH] D138418: [LoongArch] Add remaining intrinsics for CRC check instructions

SixWeining added inline comments. Comment at: llvm/lib/Target/LoongArch/LoongArchISelLowering.cpp:925 llvm_unreachable("Unexpected Intrinsic."); -case Intrinsic::loongarch_crc_w_d_w: { - Results.push_back(DAG.getNode( - ISD::TRUNCATE, DL, N->getValueType(

[PATCH] D138489: [tsan] Add tsan support for loongarch64

SixWeining added inline comments. Comment at: compiler-rt/lib/sanitizer_common/sanitizer_linux.cpp:1543 +"r"(__fn), "r"(__arg), "r"(nr_clone), "i"(__NR_exit) + : "memory"); + return res; tangyouling wrote: > SixWeining wrote: > > Shall we list $t0-$

[PATCH] D138489: [tsan] Add tsan support for loongarch64

SixWeining added inline comments. Comment at: compiler-rt/lib/sanitizer_common/sanitizer_linux.cpp:1543 +"r"(__fn), "r"(__arg), "r"(nr_clone), "i"(__NR_exit) + : "memory"); + return res; xry111 wrote: > xry111 wrote: > > SixWeining wrote: > > > tang

[PATCH] D138418: [LoongArch] Add remaining intrinsics for CRC check instructions

SixWeining added inline comments. Comment at: llvm/lib/Target/LoongArch/LoongArchISelLowering.cpp:925 llvm_unreachable("Unexpected Intrinsic."); -case Intrinsic::loongarch_crc_w_d_w: { - Results.push_back(DAG.getNode( - ISD::TRUNCATE, DL, N->getValueType(

[PATCH] D139177: [LoongArch] Specify registers used for exception handling

SixWeining created this revision. SixWeining added reviewers: xen0n, xry111, MaskRay, wangleiat. Herald added a subscriber: StephenFan. Herald added a project: All. SixWeining requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. See definition i

[PATCH] D139177: [LoongArch] Specify registers used for exception handling

SixWeining updated this revision to Diff 479553. SixWeining added a comment. Add clang test. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D139177/new/ https://reviews.llvm.org/D139177 Files: clang/lib/Basic/Targets/LoongArch.h clang/test/CodeG

[PATCH] D141785: [Clang][LoongArch] Implement patchable function entry

This revision was automatically updated to reflect the committed changes. Closed by commit rGdb5dfec9d444: [Clang][LoongArch] Implement patchable function entry (authored by WANG Xuerui , committed by SixWeining). Changed prior to commit: https://reviews.llvm.org/D141785?vs=49

[PATCH] D142688: [Clang][Driver] Handle LoongArch multiarch tuples

SixWeining added inline comments. Comment at: clang/lib/Driver/ToolChains/Arch/LoongArch.cpp:66 + default: +return IsLA32 ? "ilp32d" : "lp64d"; + } Better to be `f`? (Probably most 32-bit hardwares do not support double-float? But I'm not sure about this.)

[PATCH] D153707: [Clang][LoongArch] Consume and check -mabi and -mfpu even if -m*-float is present

SixWeining accepted this revision. SixWeining added a comment. This revision is now accepted and ready to land. LGTM. Thanks. Comment at: clang/lib/Driver/ToolChains/Arch/LoongArch.cpp:64 +// conflicts with the higher-priority settings implied by -m*-float. +if (!Implie

[PATCH] D152671: [doc][LoongArch] Add missed release note about `ual` feature addition

SixWeining added inline comments. Comment at: llvm/docs/ReleaseNotes.rst:163 * The ``lp64s`` ABI is supported now and has been tested on Rust bare-matal target. +* An target feature ``ual`` is introduced to allow unaligned memory accesses and + this feature is enabled by defa

[PATCH] D152671: [doc][LoongArch] Add missed release note about `ual` feature addition

SixWeining updated this revision to Diff 534404. SixWeining added a comment. Address xen0n's comments. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D152671/new/ https://reviews.llvm.org/D152671 Files: clang/docs/ReleaseNotes.rst llvm/docs/Rele

[PATCH] D152671: [doc][LoongArch] Add missed release note about `ual` feature addition

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGd0a32b0273b9: [doc][LoongArch] Add missed release note about `ual` feature addition (authored by SixWeining). Repository: rG LLVM Github Monorepo

[PATCH] D153707: [Clang][LoongArch] Consume and check -mabi and -mfpu even if -m*-float is present

This revision was automatically updated to reflect the committed changes. Closed by commit rGf6932007ab4e: [Clang][LoongArch] Consume and check -mabi and -mfpu even if -m*-float is… (authored by WANG Xuerui , committed by SixWeining). Repository: rG LLVM Github Monorepo CHANG

[PATCH] D140727: [XRay] Add initial support for loongarch64

SixWeining added a comment. In D140727#4019019 , @SixWeining wrote: > D140725 is abandoned. Let me defer this > change until we support 64bit PC-relative relocation for `SymA - SymB`. Update: R_LARCH_64_PCREL can be e

[PATCH] D143806: [Clang][Test] Add llvm-lto, llvm-lto2 and llvm-profdata to the tool substitutions list

SixWeining created this revision. SixWeining added reviewers: dblaikie, jansvoboda11. Herald added a subscriber: inglorion. Herald added a project: All. SixWeining requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Similar to issue fixed in D1

[PATCH] D143806: [Clang][Test] Add llvm-lto, llvm-lto2 and llvm-profdata to the tool substitutions list

This revision was automatically updated to reflect the committed changes. Closed by commit rG82c6e90d6a3d: [Clang][Test] Add llvm-lto, llvm-lto2 and llvm-profdata to the tool… (authored by SixWeining). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D14

[PATCH] D141785: [Clang][LoongArch] Implement patchable function entry

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

[PATCH] D149946: [LoongArch] Define `ual` feature and override `allowsMisalignedMemoryAccesses`

SixWeining created this revision. SixWeining added reviewers: xen0n, xry111, MaskRay, rengolin, benshi001, wangleiat. Herald added subscribers: hiraditya, kristof.beyls. Herald added a project: All. SixWeining requested review of this revision. Herald added projects: clang, LLVM. Herald added subs

[PATCH] D149946: [LoongArch] Define `ual` feature and override `allowsMisalignedMemoryAccesses`

SixWeining added a comment. TODO: update llvm/clang release notes. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D149946/new/ https://reviews.llvm.org/D149946 ___ cfe-commits mailing list cfe-commits@list

[PATCH] D149946: [LoongArch] Define `ual` feature and override `allowsMisalignedMemoryAccesses`

SixWeining updated this revision to Diff 520014. SixWeining added a comment. Add a test to show related options remain unused on a target that does not support them. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D149946/new/ https://reviews.llvm.or

[PATCH] D149946: [LoongArch] Define `ual` feature and override `allowsMisalignedMemoryAccesses`

SixWeining added a comment. Thanks @rengolin and @peter.smith. I just added a new test to show options remain unused on a target that does not support them. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D149946/new/ https://reviews.llvm.org/D149946

[PATCH] D149946: [LoongArch] Define `ual` feature and override `allowsMisalignedMemoryAccesses`

SixWeining added a comment. In D149946#4324803 , @xen0n wrote: > From a LoongArch developer's perspective, it may be better to only enable UAL > for LA464 and other supporting models, instead of for the generic > `loongarch64` model too. This is because

[PATCH] D150089: [LoongArch] Support fcc* (condition flag) registers in inlineasm clobbers

SixWeining created this revision. Herald added a project: All. SixWeining requested review of this revision. Herald added projects: clang, LLVM. Herald added subscribers: llvm-commits, cfe-commits. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D150089 Files: clang/lib/Basic/Ta

[PATCH] D150089: [LoongArch] Support fcc* (condition flag) registers in inlineasm clobbers

This revision was automatically updated to reflect the committed changes. Closed by commit rG161716a7132d: [LoongArch] Support fcc* (condition flag) registers in inlineasm clobbers (authored by SixWeining). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.or

[PATCH] D136436: [Clang][LoongArch] Add register alias handling without `$` prefix

SixWeining updated this revision to Diff 521541. SixWeining added a comment. Only allow non-prefixed names for GPRs. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D136436/new/ https://reviews.llvm.org/D136436 Files: clang/lib/Basic/Targets/LoongA

[PATCH] D136436: [Clang][LoongArch] Add GPR alias handling without `$` prefix

This revision was automatically updated to reflect the committed changes. Closed by commit rG0bbf3ddf5fea: [Clang][LoongArch] Add GPR alias handling without `$` prefix (authored by SixWeining). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D136436/new

[PATCH] D150537: [Clang][LoongArch] Pass the -mabi and -target-abi options to as and cc1as respectively

SixWeining created this revision. SixWeining added reviewers: xen0n, xry111, MaskRay, hev, wangleiat. Herald added a project: All. SixWeining requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. This change is necessary to set correct EFlags acc

[PATCH] D150537: [Clang][LoongArch] Pass the -mabi and -target-abi options to as and cc1as respectively

This revision was automatically updated to reflect the committed changes. Closed by commit rGddb27b187069: [Clang][LoongArch] Pass the -mabi and -target-abi options to as and cc1as… (authored by SixWeining). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.o

[PATCH] D142688: [Clang][Driver] Handle LoongArch multiarch tuples

SixWeining added inline comments. Comment at: clang/test/Driver/linux-header-search.cpp:247 // +// Check header search on Debian loong64 +// RUN: %clang -### %s -fsyntax-only 2>&1 \ Better to use “loongarch64” when talking about debian? Refer https://wiki.debia

[PATCH] D142688: [Clang][Driver] Handle LoongArch multiarch tuples

SixWeining accepted this revision. SixWeining added inline comments. This revision is now accepted and ready to land. Comment at: clang/test/Driver/linux-header-search.cpp:247 // +// Check header search on Debian loong64 +// RUN: %clang -### %s -fsyntax-only 2>&1 \ -

[PATCH] D142688: [Clang][Driver] Handle LoongArch multiarch tuples

This revision was automatically updated to reflect the committed changes. Closed by commit rG61fee67cd77a: [Clang][Driver] Handle LoongArch multiarch tuples (authored by WANG Xuerui , committed by SixWeining). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION ht

[PATCH] D136436: [Clang][LoongArch] Add register alias handling without `$` prefix

SixWeining added a comment. In D136436#4314612 , @xen0n wrote: > Hi, any update on this? Given the `$`-less style has been long established at > this time, and present in released GCC versions, it seems we indeed have to > follow suit (and later make GC

[PATCH] D149946: [LoongArch] Define `ual` feature and override `allowsMisalignedMemoryAccesses`

SixWeining updated this revision to Diff 529164. SixWeining added a comment. rebase Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D149946/new/ https://reviews.llvm.org/D149946 Files: clang/include/clang/Driver/Options.td clang/lib/Driver/ToolCh

[PATCH] D149946: [LoongArch] Define `ual` feature and override `allowsMisalignedMemoryAccesses`

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG47601815ec3a: [LoongArch] Define `ual` feature and override `allowsMisalignedMemoryAccesses` (authored by SixWeining). Repository: rG LLVM Github

[PATCH] D152671: [doc][LoongArch] Add missed release note about `ual` feature addition

SixWeining created this revision. SixWeining added reviewers: xen0n, MaskRay, xry111, peter.smith, wangleiat. Herald added a project: All. SixWeining requested review of this revision. Herald added projects: clang, LLVM. Herald added subscribers: llvm-commits, cfe-commits. I meant to fold this int

[PATCH] D139177: [LoongArch] Specify registers used for exception handling

This revision was automatically updated to reflect the committed changes. Closed by commit rG47edc7086692: [LoongArch] Specify registers used for exception handling (authored by SixWeining). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D139177/new/

[PATCH] D138489: [tsan] Add tsan support for loongarch64

SixWeining accepted this revision. SixWeining added a comment. This revision is now accepted and ready to land. LGTM from the LoongArch side but it's better to wait for others. Comment at: compiler-rt/lib/sanitizer_common/sanitizer_linux_libcdep.cpp:270 +#elif SANITIZER_LOONGAR

[PATCH] D139288: [Clang][LoongArch] Add intrinsic for csrrd, csrwr and csrxchg

SixWeining added inline comments. Comment at: clang/lib/Headers/larchintrin.h:67-96 #define __break(/*ui15*/ _1) __builtin_loongarch_break((_1)) #define __dbar(/*ui15*/ _1) __builtin_loongarch_dbar((_1)) #define __ibar(/*ui15*/ _1) __builtin_loongarch_ibar((_1)) #define

[PATCH] D139288: [Clang][LoongArch] Add intrinsic for csrrd, csrwr and csrxchg

SixWeining accepted this revision. SixWeining 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/D139288/new/ https://reviews.llvm.org/D139288 __

[PATCH] D139397: [LoongArch] Add testcases for privileged intrinsic macros

SixWeining accepted this revision. SixWeining 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/D139397/new/ https://reviews.llvm.org/D139397 __

[PATCH] D138489: [tsan] Add tsan support for loongarch64

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGb89b42b31c45: [tsan] Add tsan support for loongarch64 (authored by tangyouling, committed by SixWeining). Repository: rG LLVM Github Monorepo CHA

[PATCH] D139686: [lsan] Add lsan support for loongarch64

SixWeining added inline comments. Comment at: compiler-rt/test/lsan/TestCases/swapcontext.cpp:8 // Missing 'getcontext' and 'makecontext' on Android. -// UNSUPPORTED: arm,aarch64,powerpc64,android +// UNSUPPORTED: arm,aarch64,loongarch64,powerpc64,android Why

[PATCH] D139612: [Clang][LoongArch] Add intrinsic for iocsrrd and iocsrwr

SixWeining accepted this revision. SixWeining added a comment. LGTM. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D139612/new/ https://reviews.llvm.org/D139612 ___ cfe-commits mailing list cfe-commits@li

[PATCH] D114505: [clang][unittests] Fix a clang unittest linking issue

SixWeining added a comment. In D114505#3181637 , @dexonsmith wrote: > LGTM. Sorry for missing this before. Looks to me like the debian build > failure above is unrelated. > > If you need someone to commit for you, please include the info for > GIT_AUTH

[PATCH] D114505: [clang][unittests] Fix a clang unittest linking issue

SixWeining added a comment. ping CHANGES SINCE LAST ACTION https://reviews.llvm.org/D114505/new/ https://reviews.llvm.org/D114505 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D114505: [clang][unittests] Fix a clang unittest linking issue

SixWeining abandoned this revision. SixWeining added a comment. Abandon because https://reviews.llvm.org/D115580 has already fixed it although later than my fix. Thanks @dexonsmith CHANGES SINCE LAST ACTION https://reviews.llvm.org/D114505/new/ https://reviews.llvm.org/D114505

<    1   2