Page MenuHomePhabricator
People saghir

saghir (Ahsan Saghir)
User

Projects

User does not belong to any projects.

User Details

User Since
Nov 15 2018, 1:02 PM (125 w, 4 d)

Recent Activity
View All

Mar 8 2021

saghir committed rGacce401068e7: [PowerPC] Change target data layout for 16-byte stack alignment (authored by saghir).
[PowerPC] Change target data layout for 16-byte stack alignment
Mar 8 2021, 6:13 AM
saghir closed D96265: [PowerPC] Change target data layout for 16-byte stack alignment.
Mar 8 2021, 6:13 AM · Restricted Project, Restricted Project

Feb 12 2021

saghir updated the diff for D96265: [PowerPC] Change target data layout for 16-byte stack alignment.

Updated tests to add run line for llc and target datalayout.

Feb 12 2021, 12:39 PM · Restricted Project, Restricted Project
saghir updated the diff for D96265: [PowerPC] Change target data layout for 16-byte stack alignment.

Merged tests into one file.

Feb 12 2021, 6:20 AM · Restricted Project, Restricted Project

Feb 8 2021

saghir added reviewers for D96265: [PowerPC] Change target data layout for 16-byte stack alignment: Restricted Project, nemanjai, jsji.
Feb 8 2021, 7:32 AM · Restricted Project, Restricted Project
saghir requested review of D96265: [PowerPC] Change target data layout for 16-byte stack alignment.
Feb 8 2021, 7:31 AM · Restricted Project, Restricted Project

Jan 29 2021

saghir committed rGcba2552bfec1: [llvm-jitlink] Replace use of deprecated gethostbyname by getaddrinfo. (authored by rzurob).
[llvm-jitlink] Replace use of deprecated gethostbyname by getaddrinfo.
Jan 29 2021, 1:11 AM
saghir closed D95477: [llvm-jitlink] Replace use of deprecated gethostbyname by getaddrinfo..
Jan 29 2021, 1:11 AM · Restricted Project

Jan 26 2021

saghir added inline comments to D95477: [llvm-jitlink] Replace use of deprecated gethostbyname by getaddrinfo..
Jan 26 2021, 2:20 PM · Restricted Project
saghir updated the diff for D95477: [llvm-jitlink] Replace use of deprecated gethostbyname by getaddrinfo..

Address comments:
Changed addrinfo Hints = {}; to addrinfo Hints{};

Jan 26 2021, 2:18 PM · Restricted Project
saghir added a reviewer for D95477: [llvm-jitlink] Replace use of deprecated gethostbyname by getaddrinfo.: sfertile.
Jan 26 2021, 1:15 PM · Restricted Project
saghir added a reviewer for D95477: [llvm-jitlink] Replace use of deprecated gethostbyname by getaddrinfo.: Xiangling_L.
Jan 26 2021, 1:14 PM · Restricted Project
saghir updated the summary of D95477: [llvm-jitlink] Replace use of deprecated gethostbyname by getaddrinfo..
Jan 26 2021, 1:07 PM · Restricted Project
saghir requested review of D95477: [llvm-jitlink] Replace use of deprecated gethostbyname by getaddrinfo..
Jan 26 2021, 1:04 PM · Restricted Project

Jan 20 2021

saghir accepted D95004: [PowerPC] Add missing negate for VPERMXOR on little endian subtargets.

LGTM.

Jan 20 2021, 12:15 AM · Restricted Project

Dec 15 2020

saghir accepted D91819: [PowerPC] Enable paired vector type and intrinsics when MMA is disabled.

LGTM.

Dec 15 2020, 7:07 AM · Restricted Project, Restricted Project, Unknown Object (Project)
saghir accepted D91974: [PowerPC] Rename the vector pair intrinsics and builtins to replace the _mma_ prefix by _vsx_.

LGTM. A minor nit regarding an incomplete comment in llvm/test/CodeGen/PowerPC/paired-vector-intrinsics.ll which can be addressed when committing the patch.

Dec 15 2020, 6:57 AM · Restricted Project, Restricted Project, Unknown Object (Project)

Dec 4 2020

saghir added a comment to D92428: [sanitizer] Make DTLS_on_tls_get_addr signal safer.
In D92428#2428496, @saghir wrote:
In D92428#2428279, @saghir wrote:

This test case is causing failure on PowerPC buildbot http://lab.llvm.org:8011/#/builders/57/builds/1924/steps/7/logs/XPASS__SanitizerCommon-tsan-powerpc64le-Linux__res:
Unexpectedly Passed Tests (1):

SanitizerCommon-tsan-powerpc64le-Linux :: Linux/resize_tls_dynamic.cpp

I have marked the test case as unsupported (by commit: 5045b831a3b9c364d6ef0f09c7a773ca451ae1cc) on all powerpc64 till this can be investigated since it was failing due to XFAIL on the clang-ppc64le-rhel buildbot. FYI @vitalybuka

Dec 4 2020, 11:23 AM · Restricted Project

Dec 3 2020

saghir accepted D91391: [PowerPC] Fix for excessive ACC copies due to PHI nodes.

LGTM.

Dec 3 2020, 6:38 AM · Restricted Project

Dec 2 2020

saghir added a comment to D92428: [sanitizer] Make DTLS_on_tls_get_addr signal safer.
In D92428#2428279, @saghir wrote:

This test case is causing failure on PowerPC buildbot http://lab.llvm.org:8011/#/builders/57/builds/1924/steps/7/logs/XPASS__SanitizerCommon-tsan-powerpc64le-Linux__res:
Unexpectedly Passed Tests (1):

SanitizerCommon-tsan-powerpc64le-Linux :: Linux/resize_tls_dynamic.cpp
Dec 2 2020, 7:17 AM · Restricted Project
saghir committed rG5045b831a3b9: [PowerPC] Mark sanitizer test case unsupported for powerpc64 (authored by saghir).
[PowerPC] Mark sanitizer test case unsupported for powerpc64
Dec 2 2020, 7:05 AM
saghir added a comment to D92428: [sanitizer] Make DTLS_on_tls_get_addr signal safer.

This test case is causing failure on PowerPC buildbot http://lab.llvm.org:8011/#/builders/57/builds/1924/steps/7/logs/XPASS__SanitizerCommon-tsan-powerpc64le-Linux__res:
Unexpectedly Passed Tests (1):

SanitizerCommon-tsan-powerpc64le-Linux :: Linux/resize_tls_dynamic.cpp
Dec 2 2020, 5:10 AM · Restricted Project

Nov 11 2020

saghir committed rG37c4ac854553: [PowerPC] Accumulator/Unprimed Accumulator register copy, spill and restore (authored by bsaleil).
[PowerPC] Accumulator/Unprimed Accumulator register copy, spill and restore
Nov 11 2020, 2:24 PM
saghir closed D90616: [PowerPC] Accumulator/Unprimed Accumulator register copy, spill and restore.
Nov 11 2020, 2:24 PM · Restricted Project
saghir accepted D90799: [PowerPC] Add paired vector load and store builtins and intrinsics.

LGTM.

Nov 11 2020, 6:50 AM · Unknown Object (Project), Restricted Project, Restricted Project

Nov 2 2020

saghir accepted D90645: [zorg][PowerPC] Don't checkout compiler-rt for runtime build.

Thanks, LGTM.

Nov 2 2020, 1:51 PM
saghir updated the summary of D90616: [PowerPC] Accumulator/Unprimed Accumulator register copy, spill and restore.
Nov 2 2020, 10:34 AM · Restricted Project
saghir accepted D90609: [zorg][PowerPC] Switch RHEL buildbot to use runtime build.

LGTM.

Nov 2 2020, 9:06 AM
saghir added a reviewer for D90616: [PowerPC] Accumulator/Unprimed Accumulator register copy, spill and restore: Restricted Project.
Nov 2 2020, 7:57 AM · Restricted Project
saghir requested review of D90616: [PowerPC] Accumulator/Unprimed Accumulator register copy, spill and restore.
Nov 2 2020, 7:56 AM · Restricted Project

Oct 23 2020

saghir committed rGedb27912a3ec: [PowerPC] Add intrinsics for MMA (authored by bsaleil).
[PowerPC] Add intrinsics for MMA
Oct 23 2020, 11:16 AM
saghir closed D89345: [PowerPC] Add intrinsics for MMA.
Oct 23 2020, 11:16 AM · Unknown Object (Project), Restricted Project
saghir updated the summary of D89345: [PowerPC] Add intrinsics for MMA.
Oct 23 2020, 7:30 AM · Unknown Object (Project), Restricted Project

Oct 15 2020

saghir committed rZORG2fc811d0946a: [zorg] [PowerPC] Limit number of threads to 20 on clang-ppc64le-rhel buildbot (authored by saghir).
[zorg] [PowerPC] Limit number of threads to 20 on clang-ppc64le-rhel buildbot
Oct 15 2020, 12:51 PM
saghir closed D88746: [zorg] [PowerPC] Limit number of threads to 20 on clang-ppc64le-rhel buildbot.
Oct 15 2020, 12:51 PM · Restricted Project
saghir retitled D88746: [zorg] [PowerPC] Limit number of threads to 20 on clang-ppc64le-rhel buildbot from [zorg] [PowerPC] Limit number of threads to 32 on clang-ppc64le-rhel buildbot to [zorg] [PowerPC] Limit number of threads to 20 on clang-ppc64le-rhel buildbot.
Oct 15 2020, 12:48 PM · Restricted Project
saghir updated the diff for D88746: [zorg] [PowerPC] Limit number of threads to 20 on clang-ppc64le-rhel buildbot.

Updating number of threads to 20.

Oct 15 2020, 12:47 PM · Restricted Project

Oct 13 2020

saghir added reviewers for D89345: [PowerPC] Add intrinsics for MMA: Restricted Project, bsaleil.
Oct 13 2020, 2:51 PM · Unknown Object (Project), Restricted Project
saghir requested review of D89345: [PowerPC] Add intrinsics for MMA.
Oct 13 2020, 2:50 PM · Unknown Object (Project), Restricted Project
saghir committed rGf3202b30b8e6: [PowerPC] Add assemble disassemble intrinsics for MMA (authored by saghir).
[PowerPC] Add assemble disassemble intrinsics for MMA
Oct 13 2020, 11:22 AM
saghir closed D88739: [PowerPC] Add assemble disassemble intrinsics for MMA.
Oct 13 2020, 11:22 AM · Restricted Project

Oct 2 2020

saghir added reviewers for D88746: [zorg] [PowerPC] Limit number of threads to 20 on clang-ppc64le-rhel buildbot: Restricted Project, stefanp, lei.
Oct 2 2020, 10:28 AM · Restricted Project
saghir requested review of D88746: [zorg] [PowerPC] Limit number of threads to 20 on clang-ppc64le-rhel buildbot.
Oct 2 2020, 10:27 AM · Restricted Project
saghir updated the diff for D88739: [PowerPC] Add assemble disassemble intrinsics for MMA.

Formatting change

Oct 2 2020, 10:07 AM · Restricted Project
saghir added reviewers for D88739: [PowerPC] Add assemble disassemble intrinsics for MMA: bsaleil, Restricted Project.
Oct 2 2020, 7:53 AM · Restricted Project
saghir requested review of D88739: [PowerPC] Add assemble disassemble intrinsics for MMA.
Oct 2 2020, 7:52 AM · Restricted Project

Sep 30 2020

saghir committed rG66d2e3f49594: [PowerPC] Add outer product instructions for MMA (authored by saghir).
[PowerPC] Add outer product instructions for MMA
Sep 30 2020, 4:07 PM
saghir closed D88043: [PowerPC] Add outer product instructions for MMA.
Sep 30 2020, 4:07 PM · Restricted Project
saghir committed rZORGc2cc01b14ef8: [zorg] [PowerPC] Limit number of threads to 64 on clang-ppc64le-rhel buildbot (authored by saghir).
[zorg] [PowerPC] Limit number of threads to 64 on clang-ppc64le-rhel buildbot
Sep 30 2020, 9:24 AM
saghir closed D88586: [zorg] [PowerPC] Limit number of threads to 64 on clang-ppc64le-rhel buildbot.
Sep 30 2020, 9:24 AM · Restricted Project
saghir added reviewers for D88586: [zorg] [PowerPC] Limit number of threads to 64 on clang-ppc64le-rhel buildbot: stefanp, nemanjai, Restricted Project.
Sep 30 2020, 9:09 AM · Restricted Project
saghir requested review of D88586: [zorg] [PowerPC] Limit number of threads to 64 on clang-ppc64le-rhel buildbot.
Sep 30 2020, 9:07 AM · Restricted Project

Sep 29 2020

saghir updated the diff for D88043: [PowerPC] Add outer product instructions for MMA.

Fixed typos and indentation.

Sep 29 2020, 12:11 PM · Restricted Project
saghir updated the diff for D88043: [PowerPC] Add outer product instructions for MMA.

Rebased after dependant patches were committed.

Sep 29 2020, 7:46 AM · Restricted Project

Sep 23 2020

saghir closed D87993: [zorg] [PowerPC] set lld as the default linker on ppc64le-clang-rhel bot.
Sep 23 2020, 7:01 AM · Restricted Project
saghir committed rZORGf6a1fc1366bd: [zorg] [PowerPC] set lld as the default linker on ppc64le-clang-rhel bot (authored by saghir).
[zorg] [PowerPC] set lld as the default linker on ppc64le-clang-rhel bot
Sep 23 2020, 7:01 AM

Sep 22 2020

saghir added a comment to D87921: Fix -funique-internal-linkage-names to work with -O2 and new pass manager.
In D87921#2288096, @morehouse wrote:

Thanks for looking. Indeed, it looks like an issue with the disk being full on the bot.

Sep 22 2020, 11:15 AM · Restricted Project

Sep 21 2020

saghir updated the summary of D88043: [PowerPC] Add outer product instructions for MMA.
Sep 21 2020, 1:53 PM · Restricted Project
saghir updated the summary of D88043: [PowerPC] Add outer product instructions for MMA.
Sep 21 2020, 1:52 PM · Restricted Project
saghir retitled D88043: [PowerPC] Add outer product instructions for MMA from [PowerPC] Add instruction definitions for MMA to [PowerPC] Add outer product instructions for MMA.
Sep 21 2020, 1:52 PM · Restricted Project
saghir requested review of D88043: [PowerPC] Add outer product instructions for MMA.
Sep 21 2020, 1:49 PM · Restricted Project

Sep 20 2020

saghir added reviewers for D87993: [zorg] [PowerPC] set lld as the default linker on ppc64le-clang-rhel bot: stefanp, Restricted Project.
Sep 20 2020, 3:09 PM · Restricted Project
saghir requested review of D87993: [zorg] [PowerPC] set lld as the default linker on ppc64le-clang-rhel bot.
Sep 20 2020, 3:08 PM · Restricted Project

Sep 9 2020

saghir abandoned D87330: [zorg] [PowerPC] Limit number of threads to 256 on ppc64le-sanitizer bot.

Since significantly different changes are needed to make this work, another patch will be posted for review, closing this one.

Sep 9 2020, 11:15 AM · Restricted Project
saghir added a comment to D87330: [zorg] [PowerPC] Limit number of threads to 256 on ppc64le-sanitizer bot.

Thank you for the inputs everyone. I will take a look at the changes needed to be done to make this work. Thanks!

Sep 9 2020, 7:02 AM · Restricted Project

Sep 8 2020

saghir added inline comments to D87330: [zorg] [PowerPC] Limit number of threads to 256 on ppc64le-sanitizer bot.
Sep 8 2020, 2:56 PM · Restricted Project
saghir added a reviewer for D87330: [zorg] [PowerPC] Limit number of threads to 256 on ppc64le-sanitizer bot: zacharyselk.
Sep 8 2020, 2:48 PM · Restricted Project
saghir updated the summary of D87330: [zorg] [PowerPC] Limit number of threads to 256 on ppc64le-sanitizer bot.
Sep 8 2020, 2:42 PM · Restricted Project
saghir updated the diff for D87330: [zorg] [PowerPC] Limit number of threads to 256 on ppc64le-sanitizer bot.

fix indentation

Sep 8 2020, 2:40 PM · Restricted Project
saghir requested review of D87330: [zorg] [PowerPC] Limit number of threads to 256 on ppc64le-sanitizer bot.
Sep 8 2020, 2:32 PM · Restricted Project

Sep 2 2020

saghir committed rGa27398a8151d: [PowerPC] Update MemorySanitizer test to cater for number of CPUs > 1024 (authored by saghir).
[PowerPC] Update MemorySanitizer test to cater for number of CPUs > 1024
Sep 2 2020, 1:29 PM
saghir closed D87053: [PowerPC] Update MemorySanitizer test to cater for number of CPUs > 1024.
Sep 2 2020, 1:29 PM · Restricted Project
saghir added a reviewer for D87053: [PowerPC] Update MemorySanitizer test to cater for number of CPUs > 1024: jsji.
Sep 2 2020, 1:19 PM · Restricted Project
saghir added reviewers for D87053: [PowerPC] Update MemorySanitizer test to cater for number of CPUs > 1024: Restricted Project, nemanjai.
Sep 2 2020, 12:54 PM · Restricted Project
saghir requested review of D87053: [PowerPC] Update MemorySanitizer test to cater for number of CPUs > 1024.
Sep 2 2020, 12:51 PM · Restricted Project

Jul 28 2020

saghir committed rG811828a01d64: [PowerPC] Mark allocator_oom_test.cpp unsupported on PowerPC (authored by saghir).
[PowerPC] Mark allocator_oom_test.cpp unsupported on PowerPC
Jul 28 2020, 12:23 PM
saghir closed D84786: [PowerPC] Mark allocator_oom_test.cpp unsupported on PowerPC.
Jul 28 2020, 12:23 PM · Restricted Project, Unknown Object (Project), Restricted Project
saghir added a reviewer for D84786: [PowerPC] Mark allocator_oom_test.cpp unsupported on PowerPC: hfinkel.
Jul 28 2020, 12:00 PM · Restricted Project, Unknown Object (Project), Restricted Project
saghir added reviewers for D84786: [PowerPC] Mark allocator_oom_test.cpp unsupported on PowerPC: Restricted Project, nemanjai.
Jul 28 2020, 11:58 AM · Restricted Project, Unknown Object (Project), Restricted Project
saghir requested review of D84786: [PowerPC] Mark allocator_oom_test.cpp unsupported on PowerPC.
Jul 28 2020, 11:57 AM · Restricted Project, Unknown Object (Project), Restricted Project

Jun 23 2020

saghir committed rGf4c337ab85c0: [PowerPC] Add support for vector bool __int128 for Power10 (authored by saghir).
[PowerPC] Add support for vector bool __int128 for Power10
Jun 23 2020, 7:27 PM
saghir closed D81816: [PowerPC] Add support for vector bool __int128 for Power10.
Jun 23 2020, 7:27 PM · Restricted Project, Unknown Object (Project), Restricted Project

Jun 22 2020

saghir added inline comments to D81816: [PowerPC] Add support for vector bool __int128 for Power10.
Jun 22 2020, 4:07 PM · Restricted Project, Unknown Object (Project), Restricted Project

Jun 18 2020

saghir updated the diff for D81816: [PowerPC] Add support for vector bool __int128 for Power10.

Updated comment.

Jun 18 2020, 4:27 PM · Restricted Project, Unknown Object (Project), Restricted Project
saghir updated the diff for D81816: [PowerPC] Add support for vector bool __int128 for Power10.

Updated comment to reflect the change in allowed types.

Jun 18 2020, 3:54 PM · Restricted Project, Unknown Object (Project), Restricted Project
saghir accepted D81126: [PowerPC] Fix for PC Relative call protocol.

LGTM

Jun 18 2020, 12:01 PM · Restricted Project

Jun 17 2020

saghir added inline comments to D81816: [PowerPC] Add support for vector bool __int128 for Power10.
Jun 17 2020, 3:39 PM · Restricted Project, Unknown Object (Project), Restricted Project
saghir updated the diff for D81816: [PowerPC] Add support for vector bool __int128 for Power10.

Updated tests features to check.

Jun 17 2020, 3:39 PM · Restricted Project, Unknown Object (Project), Restricted Project
saghir added inline comments to D81816: [PowerPC] Add support for vector bool __int128 for Power10.
Jun 17 2020, 1:29 PM · Restricted Project, Unknown Object (Project), Restricted Project
saghir added inline comments to D81816: [PowerPC] Add support for vector bool __int128 for Power10.
Jun 17 2020, 1:28 PM · Restricted Project, Unknown Object (Project), Restricted Project
saghir updated the diff for D81816: [PowerPC] Add support for vector bool __int128 for Power10.

Updated tests.

Jun 17 2020, 1:28 PM · Restricted Project, Unknown Object (Project), Restricted Project

Jun 16 2020

saghir committed rG37e72f47a41d: [PowerPC] Add -m[no-]power10-vector clang and llvm option (authored by saghir).
[PowerPC] Add -m[no-]power10-vector clang and llvm option
Jun 16 2020, 1:12 PM
saghir closed D80758: [PowerPC] Add -m[no-]power10-vector clang and llvm option.
Jun 16 2020, 1:12 PM · Unknown Object (Project), Restricted Project, Restricted Project
saghir updated the diff for D81816: [PowerPC] Add support for vector bool __int128 for Power10.

Updated comment for allowing __int128 for vector bool.

Jun 16 2020, 12:39 PM · Restricted Project, Unknown Object (Project), Restricted Project
saghir updated the diff for D81816: [PowerPC] Add support for vector bool __int128 for Power10.

Addressed comments.

Jun 16 2020, 11:32 AM · Restricted Project, Unknown Object (Project), Restricted Project
saghir added inline comments to D81816: [PowerPC] Add support for vector bool __int128 for Power10.
Jun 16 2020, 11:00 AM · Restricted Project, Unknown Object (Project), Restricted Project
saghir updated the diff for D81816: [PowerPC] Add support for vector bool __int128 for Power10.
Jun 16 2020, 11:00 AM · Restricted Project, Unknown Object (Project), Restricted Project

Jun 14 2020

saghir created D81816: [PowerPC] Add support for vector bool __int128 for Power10.
Jun 14 2020, 7:11 PM · Restricted Project, Restricted Project, Restricted Project

Jun 5 2020

saghir updated the diff for D80758: [PowerPC] Add -m[no-]power10-vector clang and llvm option.

Updated the diff based on latest changes in master.

Jun 5 2020, 11:45 AM · Restricted Project, Restricted Project, Restricted Project

Jun 2 2020

saghir updated the diff for D80758: [PowerPC] Add -m[no-]power10-vector clang and llvm option.

Addressed nit

Jun 2 2020, 6:40 PM · Restricted Project, Restricted Project, Restricted Project