[PATCH] D156090: [Driver][XRay][test] Mark these tests as supported on loongarch64

2023-07-24 Thread Limin Zhang via Phabricator via cfe-commits
Ami-zhang abandoned this revision. Ami-zhang added a comment. Thanks @MaskRay . 05c2c09bb1f10261f6f804a2d510c55d97b1584c improved these XRay tests. Abandon this change. Repository: rG LLVM Github Monorepo CHANGES SINCE LA

[PATCH] D156090: [Driver][XRay][test] Add support for loongarch64

2023-07-24 Thread Limin Zhang via Phabricator via cfe-commits
Ami-zhang created this revision. Ami-zhang added reviewers: SixWeining, probinson, MaskRay. Herald added a subscriber: dberris. Herald added a project: All. Ami-zhang requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Depends on D156089

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

2023-07-13 Thread Limin Zhang via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rGef33d6cbfc2d: [XRay] Add initial support for loongarch64 (authored by SixWeining, committed by Ami-zhang). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D1407

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

2023-07-12 Thread Limin Zhang via Phabricator via cfe-commits
Ami-zhang updated this revision to Diff 539836. Ami-zhang added a comment. Remove unnecessary '\' symbol in xray-attribute-instrumentation.ll Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D140727/new/ https://reviews.llvm.org/D140727 Files: clang

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

2023-07-12 Thread Limin Zhang via Phabricator via cfe-commits
Ami-zhang updated this revision to Diff 539818. Ami-zhang added a comment. Make xray-attribute-instrumentation.ll less sensitive to .Ltmp/.Lxray_fn_idx label changes Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D140727/new/ https://reviews.llvm.or

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

2023-07-12 Thread Limin Zhang via Phabricator via cfe-commits
Ami-zhang added inline comments. Comment at: llvm/test/CodeGen/LoongArch/xray-attribute-instrumentation.ll:24 +; CHECK-NEXT: .Lxray_sleds_start0: +; CHECK-NEXT: .Ltmp0: +; CHECK-NEXT: .dword .Lxray_sled_begin0-.Ltmp0 MaskRay wrote: > See `llvm/test/CodeGen/X86/xr

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

2023-07-12 Thread Limin Zhang via Phabricator via cfe-commits
Ami-zhang updated this revision to Diff 539465. Ami-zhang added a comment. Hide some symbols and using some macros in asm. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D140727/new/ https://reviews.llvm.org/D140727 Files: clang/lib/Driver/XRayArg

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

2023-07-12 Thread Limin Zhang via Phabricator via cfe-commits
Ami-zhang added inline comments. Comment at: compiler-rt/lib/xray/xray_loongarch64.cpp:30 +// are 2RI12-type and 2RI16-type. +inline static uint32_t +encodeInstruction2RIx(uint32_t Opcode, uint32_t Rd, uint32_t Rj, MaskRay wrote: > Early xray code unfortunately d

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

2023-07-11 Thread Limin Zhang via Phabricator via cfe-commits
Ami-zhang updated this revision to Diff 538948. Ami-zhang added a comment. Addressed @xen0n's comments. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D140727/new/ https://reviews.llvm.org/D140727 Files: clang/lib/Driver/XRayArgs.cpp compiler-rt

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

2023-07-11 Thread Limin Zhang via Phabricator via cfe-commits
Ami-zhang added inline comments. Comment at: compiler-rt/lib/xray/xray_loongarch64.cpp:22-25 + RN_T0 = 0xC, + RN_T1 = 0xD, + RN_RA = 0x1, + RN_SP = 0x3, xen0n wrote: > I think people usually just declare the register ABI names with decimal > numbers for read

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

2023-07-07 Thread Limin Zhang via Phabricator via cfe-commits
Ami-zhang updated this revision to Diff 538022. Ami-zhang added a comment. Rebase and make some improvements. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D140727/new/ https://reviews.llvm.org/D140727 Files: clang/lib/Driver/XRayArgs.cpp compi

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

2023-07-07 Thread Limin Zhang via Phabricator via cfe-commits
Ami-zhang added inline comments. Comment at: compiler-rt/lib/xray/xray_loongarch64.cpp:23 +enum PatchOpcodes : uint32_t { + PO_ADDID = 0x02c0, // addi.d rd, rj, imm + PO_SD = 0x29c0, // st.d rd, base, offset MaskRay wrote: > I think the `PO_` style

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

2023-07-05 Thread Limin Zhang via Phabricator via cfe-commits
Ami-zhang updated this revision to Diff 537564. Ami-zhang added a comment. (1) Rebase (2) Make some minor changes to xray-attribute-instrumentation.ll Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D140727/new/ https://reviews.llvm.org/D140727 Files

[PATCH] D140690: [dfsan] Support Linux loongarch64

2023-07-03 Thread Limin Zhang via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG280d163887ea: [dfsan] Support Linux loongarch64 (authored by Ami-zhang). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://

[PATCH] D140690: [dfsan] Support Linux loongarch64

2023-06-29 Thread Limin Zhang via Phabricator via cfe-commits
Ami-zhang updated this revision to Diff 536091. Ami-zhang retitled this revision from "[compiler-rt][dfsan] Enable loongarch64 and add test support" to "[dfsan] Support Linux loongarch64". Ami-zhang edited the summary of this revision. Ami-zhang added a comment. Herald added subscribers: llvm-comm