Maybe it's better to switch off generation of the madd.fmt by default if the problem is so wide spread? I'm concerned that AFAIK no one other architectures supported by LLVM has configuration options like the proposed LLVM_DISABLE_MIPS_MADD4.

Jul 15 2020, 4:56 AM · Restricted Project

atanasyan committed rG3a6c2a61c642: [mips] Rename FeatureMadd4 to FeatureNoMadd4. NFC (authored by wzssyqa).

[mips] Rename FeatureMadd4 to FeatureNoMadd4. NFC

Jul 15 2020, 4:40 AM

atanasyan closed D83780: MIPS: change FeatureMadd4 to FeatureNoMadd4.

Jul 15 2020, 4:40 AM · Restricted Project

Jul 14 2020

atanasyan accepted D83780: MIPS: change FeatureMadd4 to FeatureNoMadd4.

Do you have commit access?

Jul 14 2020, 9:11 AM · Restricted Project

Jul 6 2020

atanasyan accepted D83225: [llvm-readobj] - Refactor the MipsGOTParser<ELFT> to stop using report_fatal_error()..

LGTM. Thanks

Jul 6 2020, 1:18 PM · Restricted Project

Jun 19 2020

atanasyan accepted D81908: [Mips] Error if a non-immediate operand is used while an immediate is expected.

Do you have commit access?

Jun 19 2020, 11:58 AM · Restricted Project

Jun 16 2020

atanasyan added a comment to D81908: [Mips] Error if a non-immediate operand is used while an immediate is expected.

In D81908#2095355, @xiangzhai wrote:

In D81908#2095335, @atanasyan wrote:

Does this patch depend on D81919?

No!

Jun 16 2020, 6:04 AM · Restricted Project

atanasyan added a comment to D81908: [Mips] Error if a non-immediate operand is used while an immediate is expected.

Does this patch depend on D81919?

Jun 16 2020, 4:58 AM · Restricted Project

atanasyan added a comment to D81919: [MIPS64] Workaround fixup_Mips_32 for getExprOpValue.

Please include context in all patches sent to review.
To get a full diff, use one of the following commands:

git show HEAD -U999999 > mypatch.patch
git diff -U999999 @{u} > mypatch.patch
git diff HEAD~1 -U999999 > mypatch.patch

https://llvm.org/docs/Phabricator.html#requesting-a-review-via-the-web-interface

Jun 16 2020, 4:58 AM · Restricted Project

Jun 3 2020

atanasyan updated subscribers of D80392: [WIP][mips] Use pc-relative relocations in .eh_frame.

Jun 3 2020, 2:43 AM · Restricted Project

Jun 2 2020

atanasyan committed rGb00f0d4238cb: [mips] Support 64-bit relative relocations (authored by atanasyan).

[mips] Support 64-bit relative relocations

Jun 2 2020, 2:13 AM

atanasyan closed D80390: [mips] Support 64-bit relative relocations.

Jun 2 2020, 2:12 AM · Restricted Project

atanasyan added a comment to D80392: [WIP][mips] Use pc-relative relocations in .eh_frame.

I cannot select default behaviour for the LLVM in case of generating .eh_frame sections for MIPS:

Unconditionally use pc-relative relocations.
Use pc-relative relocations by default, but provide options for switching to absolute relocations.
Use absolute relocations by default, but provide options for switching to pc-relative relocations.

Jun 2 2020, 2:12 AM · Restricted Project

atanasyan updated the diff for D80390: [mips] Support 64-bit relative relocations.

Thanks for review, testing and comments.

Jun 2 2020, 1:04 AM · Restricted Project

atanasyan retitled D80390: [mips] Support 64-bit relative relocations from [WIP][mips] Support 64-bit relative relocations to [mips] Support 64-bit relative relocations.

Jun 2 2020, 1:04 AM · Restricted Project

May 22 2020

atanasyan added a comment to D80390: [mips] Support 64-bit relative relocations.

In D80390#2049724, @MaskRay wrote:

.quad foo - . is representable

ld.so supports such relocation chain.

D78590 can be fixed for MIPS

D72228 can get rid of the FDECFIEncoding = dwarf::DW_EH_PE_pcrel | dwarf::DW_EH_PE_sdata4; hack

For various relocation processing tool, it supporting a relocation chain as easy as supporting an addition relocation type (R_MIPS_PC64), by, adding case R_MIPS_PC64:?

May 22 2020, 1:13 AM · Restricted Project

atanasyan added a reviewer for D80421: [Mips] use correct ld.so for musl soft float: atanasyan.

Please include as much context as possible with your diff. This instruction helps to do that.
Such patch needs test case(s). Maybe it's enough to update the linux-ld.c test case which is failed now. Update failed cases and check both "soft" and "hard" float cases in this test.

May 22 2020, 12:01 AM · Restricted Project

May 21 2020

atanasyan committed rGc7923e7705d0: [mips] Reorganize check directives in the test. NFC (authored by atanasyan).

[mips] Reorganize check directives in the test. NFC

May 21 2020, 11:22 AM

atanasyan created D80390: [mips] Support 64-bit relative relocations.

May 21 2020, 11:21 AM · Restricted Project

atanasyan created D80392: [WIP][mips] Use pc-relative relocations in .eh_frame.

May 21 2020, 11:21 AM · Restricted Project

May 13 2020

atanasyan committed rG524457edbc3d: [mips] Fix typo in FileCheck directives - replace \0xa0 char by space. NFC (authored by atanasyan).

[mips] Fix typo in FileCheck directives - replace \0xa0 char by space. NFC

May 13 2020, 2:39 AM

May 7 2020

atanasyan accepted D78108: [CSInfo][MIPS] Describe parameter value loaded by ADDiu instruction.

Looks good overall, just a few nits inline.

May 7 2020, 11:16 PM · Restricted Project, debug-info

May 1 2020

atanasyan accepted D79270: [test] Fix lld's ELF/linkerscript/thunk-gen-mips.s.

Initially the test case just checks that LLD does not crash under some conditions. It's not mandatory to test exact symbols' values and/or offsets between symbols. But after this fix the test additionally becomes more tolerant to exact symbol addresses. It's fine.
As to me, I'd remove the "Ideally we'd use..." comment.

May 1 2020, 11:25 PM · Restricted Project

Apr 24 2020

atanasyan committed rG0eec6662f669: [MC][mips] Replace setRType## methods by single setRTypes function. NFC (authored by atanasyan).

[MC][mips] Replace setRType## methods by single setRTypes function. NFC

Apr 24 2020, 2:41 AM

Apr 3 2020

atanasyan committed rGf22445bf5760: [mips][test] Remove redundant and invalid `CHECK-NOT` directives. NFC (authored by atanasyan).

[mips][test] Remove redundant and invalid `CHECK-NOT` directives. NFC

Apr 3 2020, 6:58 AM

atanasyan added a comment to D77354: [DO NOT MERGE] added FileCheck colons that broke things.

I have fixed MIPS-related mistakes at rGf22445b. Thanks for finding them.

Apr 3 2020, 6:57 AM · Restricted Project

Mar 27 2020

atanasyan added inline comments to D76842: [Mips] Make MipsBranchExpansion::replaceBranch() aware of BBIT family of branch.

Mar 27 2020, 2:52 PM · Restricted Project

atanasyan accepted D76842: [Mips] Make MipsBranchExpansion::replaceBranch() aware of BBIT family of branch.

LGTM. Two nits inlined.

Mar 27 2020, 1:43 PM · Restricted Project

Mar 22 2020

atanasyan added a comment to D76247: [MIPS] Implement MIPS3D vector instructions..

Thanks for the catching the issues.

Mar 22 2020, 2:28 PM · Restricted Project

atanasyan committed rG2dc4eb08cd99: [mips] Implement .cpadd directive (authored by atanasyan).

[mips] Implement .cpadd directive

Mar 22 2020, 1:56 PM

atanasyan committed rG9bbddfbeaabd: [mips] Implement sne pseudo instruction (authored by atanasyan).

[mips] Implement sne pseudo instruction

Mar 22 2020, 1:56 PM

atanasyan committed rG862f120fdb0e: [mips] Remove instructions related to "wired paired single" from the P5600… (authored by atanasyan).

[mips] Remove instructions related to "wired paired single" from the P5600…

Mar 22 2020, 1:56 PM

atanasyan committed rGdca9e40c0cf8: [mips] Implement sle/sleu pseudo instructions (authored by atanasyan).

[mips] Implement sle/sleu pseudo instructions

Mar 22 2020, 1:56 PM

atanasyan committed rGecc92fd018c6: [mips] Add HasMips3D to the list of features unsupported by P5600 model. (authored by atanasyan).

[mips] Add HasMips3D to the list of features unsupported by P5600 model.

Mar 22 2020, 1:56 PM

atanasyan committed rG0f15ace018bf: [mips] Rename target feature Mips3D => HasMips3D. NFC (authored by atanasyan).

[mips] Rename target feature Mips3D => HasMips3D. NFC

Mar 22 2020, 1:56 PM

Mar 17 2020

atanasyan committed rG73b1da160513: [MIPS] Implement MIPS3D vector instructions (authored by atanasyan).

[MIPS] Implement MIPS3D vector instructions

Mar 17 2020, 7:26 AM

atanasyan closed D76247: [MIPS] Implement MIPS3D vector instructions..

Mar 17 2020, 7:26 AM · Restricted Project

atanasyan accepted D76247: [MIPS] Implement MIPS3D vector instructions..

LGTM

Mar 17 2020, 7:25 AM · Restricted Project

Mar 15 2020

atanasyan committed rGe0ab0e6a281f: [MIPS] Implement PUL.PS and PUU.PS instructions (authored by atanasyan).

[MIPS] Implement PUL.PS and PUU.PS instructions

Mar 15 2020, 11:42 PM

atanasyan closed D75812: [MIPS] Implement PUL.PS and PUU.PS instructions.

Mar 15 2020, 11:42 PM · Restricted Project

Mar 13 2020

atanasyan accepted D75812: [MIPS] Implement PUL.PS and PUU.PS instructions.

LGTM Thanks for the patch.

Mar 13 2020, 3:40 PM · Restricted Project

Mar 8 2020

atanasyan requested changes to D75812: [MIPS] Implement PUL.PS and PUU.PS instructions.

The following test cases failed on my machine. Initially, assembler shows error: unknown instruction. Now assembler shows correct error message instruction requires a CPU feature not currently enabled and the checks should be updated and moved to invalid-xxx.s files.

MC/Mips/mips1/invalid-mips5-wrong-error.s
MC/Mips/mips2/invalid-mips5-wrong-error.s
MC/Mips/mips3/invalid-mips5-wrong-error.s
MC/Mips/mips4/invalid-mips5-wrong-error.s
MC/Mips/mips64r6/invalid-mips5-wrong-error.s

Mar 8 2020, 5:50 AM · Restricted Project

Feb 27 2020

atanasyan added inline comments to D75216: [lld][ELF] Add some release notes.

Feb 27 2020, 12:21 AM · Restricted Project

Feb 26 2020

atanasyan added inline comments to D75216: [lld][ELF] Add some release notes.

Feb 26 2020, 10:08 PM · Restricted Project

Feb 16 2020

atanasyan accepted D74624: [MIPS GlobalISel] Select 4 byte unaligned load and store.

LGTM

Feb 16 2020, 10:13 PM · Restricted Project

atanasyan accepted D74625: [MIPS GlobalISel] Legalize non-power-of-2 and unaligned load and store.

LGTM

Feb 16 2020, 10:13 PM · Restricted Project

atanasyan accepted D74623: [MIPS GlobalISel] RegBankSelect G_MERGE_VALUES and G_UNMERGE_VALUES.

LGTM

Feb 16 2020, 10:13 PM · Restricted Project

Feb 10 2020

atanasyan accepted D73644: [Mips] Add intrinsics for 4-byte and 8-byte MSA loads/stores..

Looking good to me as-is.

Feb 10 2020, 4:26 AM · Restricted Project, Restricted Project

Feb 6 2020

atanasyan added a comment to D73644: [Mips] Add intrinsics for 4-byte and 8-byte MSA loads/stores..

I see, thanks. Is there the same or similar functionality in GCC?

Feb 6 2020, 8:34 AM · Restricted Project, Restricted Project

Jan 30 2020

atanasyan added a comment to D73644: [Mips] Add intrinsics for 4-byte and 8-byte MSA loads/stores..

Is it possible to emulate these new intrinsics using existing ones and some additional code? Is code generated in this case much larger/slower then the code generated by the new intrinsics?

Jan 30 2020, 5:17 AM · Restricted Project, Restricted Project