azharudd (Azharuddin Mohammed)
User

Projects

User does not belong to any projects.

User Details

User Since
Apr 28 2016, 11:11 AM (112 w, 13 h)

Recent Activity

Thu, May 31

azharudd added a comment to D47224: [cmake] Guard another instance where symlinks are being created.

I'm not sure about this patch: These commands do not install the aliases, they are created in the build directory and may be needed for testing.

Thu, May 31, 3:09 PM

Thu, May 24

azharudd committed rL333232: Revert "[cmake] [ARM] Check if VFP is supported before including any VFP….
Revert "[cmake] [ARM] Check if VFP is supported before including any VFP…
Thu, May 24, 2:42 PM
azharudd committed rCRT333232: Revert "[cmake] [ARM] Check if VFP is supported before including any VFP….
Revert "[cmake] [ARM] Check if VFP is supported before including any VFP…
Thu, May 24, 2:42 PM
azharudd added a comment to D47217: [cmake] [ARM] Check if VFP is supported before including any VFP builtins.

@peter.smith Thanks for the review.

Thu, May 24, 11:58 AM
azharudd committed rCRT333216: [cmake] [ARM] Check if VFP is supported before including any VFP builtins.
[cmake] [ARM] Check if VFP is supported before including any VFP builtins
Thu, May 24, 11:57 AM
azharudd committed rL333216: [cmake] [ARM] Check if VFP is supported before including any VFP builtins.
[cmake] [ARM] Check if VFP is supported before including any VFP builtins
Thu, May 24, 11:57 AM
azharudd closed D47217: [cmake] [ARM] Check if VFP is supported before including any VFP builtins.
Thu, May 24, 11:57 AM

Wed, May 23

azharudd updated the diff for D47217: [cmake] [ARM] Check if VFP is supported before including any VFP builtins.
Wed, May 23, 12:45 PM

May 22 2018

azharudd created D47224: [cmake] Guard another instance where symlinks are being created.
May 22 2018, 1:36 PM
azharudd created D47221: [cmake] Fix libomptarget/test/CMakeLists.txt.
May 22 2018, 1:14 PM
azharudd created D47217: [cmake] [ARM] Check if VFP is supported before including any VFP builtins.
May 22 2018, 12:35 PM

Mar 13 2018

azharudd committed rL327423: Build system changes for RISCV.
Build system changes for RISCV
Mar 13 2018, 10:07 AM
azharudd closed D44153: Build system changes for RISCV.
Mar 13 2018, 10:07 AM
azharudd added a comment to D44153: Build system changes for RISCV.
In D44153#1031969, @asb wrote:

Hi Azharuddin, many thanks for the contribution. I hadn't spotted this part of LLVM's CMake build system, and it's definitely useful to be able to build just the RISCV backend. I've tried this locally, and the problem I'm seeing is that a good number of Clang and LLVM tests fail when you don't build X86 support. On my system, 351 unexpected failures across Clang+LLVM. This is a problem with the tests rather than this patch (they should really be disabled if there is no X86 support), but it does limit the usefulness of just building _only_ the RISC-V backend. Is it expected that you build an LLVM with X86 and RISCV, ensure 100% tests pass, then build with just the RISCV backend and ignore any new failures?

The problems I'm seeing aren't introduced by this patch, so LGTM. I'd be curious on how/if you work around the tests issue though.

Mar 13 2018, 9:47 AM
azharudd added a comment to D44153: Build system changes for RISCV.
In D44153#1031976, @asb wrote:

Actually, looking again I'm not having any problems building just the RISCV backend even without this patch. e.g. an invocation like the following seems fine:

cmake -G Ninja -DCMAKE_BUILD_TYPE="Debug" \
  -DBUILD_SHARED_LIBS=True -DLLVM_USE_SPLIT_DWARF=True \
  -DLLVM_OPTIMIZED_TABLEGEN=True \
  -DLLVM_BUILD_TESTS=True \
  -DLLVM_TARGETS_TO_BUILD="RISCV" \
  -DLLVM_EXPERIMENTAL_TARGETS_TO_BUILD="RISCV" ../
cmake --build .

Could you please clarify exactly which build problems this fixes?

Mar 13 2018, 9:29 AM

Mar 6 2018

azharudd created D44153: Build system changes for RISCV.
Mar 6 2018, 7:56 AM

Jan 3 2018

azharudd committed rCRT321760: [cmake] Fix typo in test/asan/CMakeLists.txt.
[cmake] Fix typo in test/asan/CMakeLists.txt
Jan 3 2018, 1:31 PM
azharudd committed rL321760: [cmake] Fix typo in test/asan/CMakeLists.txt.
[cmake] Fix typo in test/asan/CMakeLists.txt
Jan 3 2018, 1:31 PM
azharudd closed D41214: [cmake] Fix typo in test/asan/CMakeLists.txt.
Jan 3 2018, 1:31 PM
azharudd added a comment to D41214: [cmake] Fix typo in test/asan/CMakeLists.txt.

Thanks for the review @smeenai .

Jan 3 2018, 1:21 PM
azharudd added a comment to D41214: [cmake] Fix typo in test/asan/CMakeLists.txt.

The two should be equivalent unless CMAKE_SYSTEM_VERSION expands out to a value that's a variable name (which is unlikely, since it should be expanding out to a number), but this is cleaner and more consistent.

Jan 3 2018, 1:19 PM

Jan 2 2018

azharudd added reviewers for D41214: [cmake] Fix typo in test/asan/CMakeLists.txt: smeenai, beanz.
Jan 2 2018, 12:00 PM

Dec 13 2017

azharudd updated the summary of D41214: [cmake] Fix typo in test/asan/CMakeLists.txt.
Dec 13 2017, 5:39 PM
azharudd created D41214: [cmake] Fix typo in test/asan/CMakeLists.txt.
Dec 13 2017, 5:37 PM

Nov 16 2017

azharudd committed rL318437: Fix RISCV build after r318352.
Fix RISCV build after r318352
Nov 16 2017, 10:40 AM
azharudd closed D40139: Fix RISCV build after r318352 by committing rL318437: Fix RISCV build after r318352.
Nov 16 2017, 10:39 AM
azharudd created D40139: Fix RISCV build after r318352.
Nov 16 2017, 10:22 AM

Mar 12 2017

azharudd committed rL297582: Remove CRC32 instructions from AArch64InstrInfo::hasShiftedReg.
Remove CRC32 instructions from AArch64InstrInfo::hasShiftedReg
Mar 12 2017, 7:14 AM
azharudd closed D30274: Remove CRC32 instructions from AArch64InstrInfo::hasShiftedReg by committing rL297582: Remove CRC32 instructions from AArch64InstrInfo::hasShiftedReg.
Mar 12 2017, 7:14 AM

Mar 7 2017

azharudd added a comment to D30274: Remove CRC32 instructions from AArch64InstrInfo::hasShiftedReg.

Hi, right, so Tim has approved it on the mailing list, which didn't get transferred here.

But he also said an additional interesting point why you have to use mcpu and not another mattr: this is a code path that only triggers with A53 because of its scheduler.

LGTM, too. Can you just add the scheduler info (instead of just saying it hits with A53) on the commit line?

thanks!

Mar 7 2017, 10:30 AM
azharudd added a comment to D30274: Remove CRC32 instructions from AArch64InstrInfo::hasShiftedReg.

Can you update the context around using git diff -U999?

Mar 7 2017, 10:28 AM
azharudd updated the diff for D30274: Remove CRC32 instructions from AArch64InstrInfo::hasShiftedReg.

Updated the context around the change.

Mar 7 2017, 10:27 AM
azharudd updated the summary of D30274: Remove CRC32 instructions from AArch64InstrInfo::hasShiftedReg.
Mar 7 2017, 10:26 AM

Mar 2 2017

azharudd updated the summary of D30274: Remove CRC32 instructions from AArch64InstrInfo::hasShiftedReg.
Mar 2 2017, 3:42 PM
azharudd updated the summary of D30274: Remove CRC32 instructions from AArch64InstrInfo::hasShiftedReg.
Mar 2 2017, 3:42 PM
azharudd updated the summary of D30274: Remove CRC32 instructions from AArch64InstrInfo::hasShiftedReg.
Mar 2 2017, 3:41 PM

Feb 22 2017

azharudd added a reviewer for D30274: Remove CRC32 instructions from AArch64InstrInfo::hasShiftedReg: rengolin.
Feb 22 2017, 2:35 PM
azharudd created D30274: Remove CRC32 instructions from AArch64InstrInfo::hasShiftedReg.
Feb 22 2017, 2:34 PM

Nov 7 2016

azharudd added a reviewer for D26289: [CMake] Fix llvm_setup_rpath function: sgundapa.
Nov 7 2016, 1:18 PM
azharudd added a comment to D26289: [CMake] Fix llvm_setup_rpath function.

I think there is actually a more general solution to this. At the top of the function we can just have:

if(CMAKE_INSTALL_RPATH)
  return()
endif()

Then we can remove the conditionals on CMAKE_INSTALL_RPATH, and just allow the target to inherit from the global setting, which is the default.

Nov 7 2016, 12:57 PM
azharudd updated the diff for D26289: [CMake] Fix llvm_setup_rpath function.
Nov 7 2016, 12:51 PM

Nov 3 2016

azharudd updated D26289: [CMake] Fix llvm_setup_rpath function.
Nov 3 2016, 2:03 PM
azharudd retitled D26289: [CMake] Fix llvm_setup_rpath function from to [CMake] Fix llvm_setup_rpath function.
Nov 3 2016, 2:02 PM

Oct 27 2016

azharudd added reviewers for D25994: Set default values before conditionals: hfinkel, MatzeB.
Oct 27 2016, 1:07 PM
azharudd added a reviewer for D25994: Set default values before conditionals: mgrang.
Oct 27 2016, 12:57 PM
azharudd abandoned D25743: Teach llvm-config about default sysroot..
Oct 27 2016, 11:13 AM
azharudd added a reviewer for D25967: Avoid running regex on None: mgrang.
Oct 27 2016, 11:02 AM

Oct 26 2016

azharudd retitled D25994: Set default values before conditionals from to Set default values before conditionals.
Oct 26 2016, 8:48 AM

Oct 25 2016

azharudd added a comment to D25967: Avoid running regex on None.

Chris, can you please commit this change. I do not have commit rights yet. Thanks.

Oct 25 2016, 4:47 PM
azharudd added inline comments to D25967: Avoid running regex on None.
Oct 25 2016, 2:33 PM
azharudd retitled D25967: Avoid running regex on None from to Avoid running regex on None.
Oct 25 2016, 2:29 PM
azharudd added a comment to D25743: Teach llvm-config about default sysroot..

I don't understand what you're doing here, or the justification for it. The LLVM build system has no concept of a default sysroot, and I'm not sure I understand why you're creating one. Please explain what you're trying to do and why you feel that you need this.

Oct 25 2016, 11:37 AM

Oct 18 2016

azharudd added reviewers for D25743: Teach llvm-config about default sysroot.: ddunbar, chapuni.
Oct 18 2016, 1:02 PM
azharudd retitled D25743: Teach llvm-config about default sysroot. from to Teach llvm-config about default sysroot..
Oct 18 2016, 12:56 PM
azharudd updated D25574: Fix clang tests.
Oct 18 2016, 12:27 PM

Oct 13 2016

azharudd changed the visibility for D25574: Fix clang tests.
Oct 13 2016, 1:12 PM
azharudd added a reviewer for D25574: Fix clang tests: rsmith.
Oct 13 2016, 1:11 PM
azharudd updated the diff for D25574: Fix clang tests.
Oct 13 2016, 1:07 PM
azharudd retitled D25574: Fix clang tests from to Fix clang tests.
Oct 13 2016, 1:04 PM