In D85554#2231462, @dokyungs wrote:

In D85554#2230791, @azharudd wrote:

Reverted this in 8831e34771fec4dfbe62a6e31d9bc9419a3b93c3 to get the bots green again. Please feel free to ping if you need someone to test this on macOS.

Could you try this patch again on top of https://reviews.llvm.org/D86247 which hopefully fixes the failing test on macOS?

Aug 25 2020, 12:26 PM · Restricted Project

azharudd added a comment to D85554: [libFuzzer] Fix arguments of InsertPartOf/CopyPartOf calls in CrossOver mutator..

@dokyungs, looking into this now.

Aug 25 2020, 9:50 AM · Restricted Project

Aug 21 2020

azharudd committed rG6a64079699e7: Fix llvm/test/tools/lto/hide-linkonce-odr.ll (authored by azharudd).

Fix llvm/test/tools/lto/hide-linkonce-odr.ll

Aug 21 2020, 6:33 PM

azharudd added a comment to D85554: [libFuzzer] Fix arguments of InsertPartOf/CopyPartOf calls in CrossOver mutator..

Reverted this in 8831e34771fec4dfbe62a6e31d9bc9419a3b93c3 to get the bots green again. Please feel free to ping if you need someone to test this on macOS.

Aug 21 2020, 10:04 AM · Restricted Project

azharudd added a reverting change for rGbb54bcf84970: [libFuzzer] Fix arguments of InsertPartOf/CopyPartOf calls in CrossOver mutator.: rG8831e34771fe: Revert "[libFuzzer] Fix arguments of InsertPartOf/CopyPartOf calls in CrossOver….

Aug 21 2020, 10:01 AM

azharudd committed rG8831e34771fe: Revert "[libFuzzer] Fix arguments of InsertPartOf/CopyPartOf calls in CrossOver… (authored by azharudd).

Revert "[libFuzzer] Fix arguments of InsertPartOf/CopyPartOf calls in CrossOver…

Aug 21 2020, 10:01 AM

azharudd added a reverting change for D85554: [libFuzzer] Fix arguments of InsertPartOf/CopyPartOf calls in CrossOver mutator.: rG8831e34771fe: Revert "[libFuzzer] Fix arguments of InsertPartOf/CopyPartOf calls in CrossOver….

Aug 21 2020, 10:01 AM · Restricted Project

azharudd committed rG7ed3286a512d: [LibFuzzer] [tests] [Darwin] Use the pthread library from the SDK (authored by azharudd).

[LibFuzzer] [tests] [Darwin] Use the pthread library from the SDK

Aug 21 2020, 9:26 AM

Aug 19 2020

azharudd added a comment to D85554: [libFuzzer] Fix arguments of InsertPartOf/CopyPartOf calls in CrossOver mutator..

In D85554#2227180, @dokyungs wrote:

In D85554#2226648, @azharudd wrote:

Looks like this change is causing the value-profile-load.test test to fail. Can you please take a look?

******************** TEST 'libFuzzer :: value-profile-load.test' FAILED ********************
Script:
--
: 'RUN: at line 2';     /Users/buildslave/jenkins/workspace/clang-stage1-RA/clang-build/./bin/clang  --driver-mode=g++ -O2 -gline-tables-only -fsanitize=address,fuzzer -I/Users/buildslave/jenkins/workspace/clang-stage1-RA/llvm-project/compiler-rt/lib/fuzzer -arch x86_64 -stdlib=libc++ -mmacosx-version-min=10.9 -isysroot /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.15.sdk /Users/buildslave/jenkins/workspace/clang-stage1-RA/llvm-project/compiler-rt/test/fuzzer/LoadTest.cpp -fsanitize-coverage=trace-gep -o /Users/buildslave/jenkins/workspace/clang-stage1-RA/clang-build/tools/clang/runtime/compiler-rt-bins/test/fuzzer/X86_64DefaultDarwinConfig/Output/value-profile-load.test.tmp-LoadTest
: 'RUN: at line 3';   not  /Users/buildslave/jenkins/workspace/clang-stage1-RA/clang-build/tools/clang/runtime/compiler-rt-bins/test/fuzzer/X86_64DefaultDarwinConfig/Output/value-profile-load.test.tmp-LoadTest -seed=2 -use_cmp=0 -use_value_profile=1 -runs=20000000 2>&1 | FileCheck /Users/buildslave/jenkins/workspace/clang-stage1-RA/llvm-project/compiler-rt/test/fuzzer/value-profile-load.test
--
Exit Code: 1

Command Output (stderr):
--
+ : 'RUN: at line 2'
+ /Users/buildslave/jenkins/workspace/clang-stage1-RA/clang-build/./bin/clang --driver-mode=g++ -O2 -gline-tables-only -fsanitize=address,fuzzer -I/Users/buildslave/jenkins/workspace/clang-stage1-RA/llvm-project/compiler-rt/lib/fuzzer -arch x86_64 -stdlib=libc++ -mmacosx-version-min=10.9 -isysroot /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.15.sdk /Users/buildslave/jenkins/workspace/clang-stage1-RA/llvm-project/compiler-rt/test/fuzzer/LoadTest.cpp -fsanitize-coverage=trace-gep -o /Users/buildslave/jenkins/workspace/clang-stage1-RA/clang-build/tools/clang/runtime/compiler-rt-bins/test/fuzzer/X86_64DefaultDarwinConfig/Output/value-profile-load.test.tmp-LoadTest
+ : 'RUN: at line 3'
+ not /Users/buildslave/jenkins/workspace/clang-stage1-RA/clang-build/tools/clang/runtime/compiler-rt-bins/test/fuzzer/X86_64DefaultDarwinConfig/Output/value-profile-load.test.tmp-LoadTest -seed=2 -use_cmp=0 -use_value_profile=1 -runs=20000000
+ FileCheck /Users/buildslave/jenkins/workspace/clang-stage1-RA/llvm-project/compiler-rt/test/fuzzer/value-profile-load.test
/Users/buildslave/jenkins/workspace/clang-stage1-RA/llvm-project/compiler-rt/test/fuzzer/value-profile-load.test:1:8: error: CHECK: expected string not found in input
CHECK: AddressSanitizer: global-buffer-overflow
       ^
<stdin>:1:1: note: scanning from here
INFO: Seed: 2
^
<stdin>:5:27: note: possible intended match here
INFO: A corpus is not provided, starting from an empty corpus
                          ^

Input file: <stdin>
Check file: /Users/buildslave/jenkins/workspace/clang-stage1-RA/llvm-project/compiler-rt/test/fuzzer/value-profile-load.test

-dump-input=help explains the following input dump.

Input was:
<<<<<<
           1: INFO: Seed: 2
check:1'0     X~~~~~~~~~~~~ error: no match found
           2: INFO: Loaded 1 modules (3 inline 8-bit counters): 3 [0x10af2bac8, 0x10af2bacb), 
check:1'0     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
           3: INFO: Loaded 1 PC tables (3 PCs): 3 [0x10af2bad0,0x10af2bb00), 
check:1'0     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
           4: INFO: -max_len is not provided; libFuzzer will not generate inputs larger than 4096 bytes
check:1'0     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
           5: INFO: A corpus is not provided, starting from an empty corpus
check:1'0     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
check:1'1                               ?                                   possible intended match
           6: #2 INITED cov: 2 ft: 4 corp: 1/1b exec/s: 0 rss: 31Mb
check:1'0     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
           7: #14 NEW cov: 2 ft: 5 corp: 2/4b lim: 4 exec/s: 0 rss: 31Mb L: 3/3 MS: 2 CrossOver-CopyPart-
check:1'0     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
           8: #242 NEW cov: 2 ft: 6 corp: 3/9b lim: 6 exec/s: 0 rss: 31Mb L: 5/5 MS: 3 ShuffleBytes-ShuffleBytes-CopyPart-
check:1'0     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
           9: #477 NEW cov: 3 ft: 11 corp: 4/17b lim: 8 exec/s: 0 rss: 31Mb L: 8/8 MS: 5 EraseBytes-ChangeByte-EraseBytes-InsertRepeatedBytes-CopyPart-
check:1'0     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
          10: #480 NEW cov: 3 ft: 13 corp: 5/24b lim: 8 exec/s: 0 rss: 31Mb L: 7/8 MS: 3 ChangeByte-InsertByte-CopyPart-
check:1'0     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
           .
           .
           .
>>>>>>

--

********************

Could you please apply the following patch and see if it fixes the regression?

https://reviews.llvm.org/D86247

Aug 19 2020, 2:55 PM · Restricted Project

azharudd added a comment to D85554: [libFuzzer] Fix arguments of InsertPartOf/CopyPartOf calls in CrossOver mutator..

Looks like this change is causing the value-profile-load.test test to fail. Can you please take a look?

******************** TEST 'libFuzzer :: value-profile-load.test' FAILED ********************
Script:
--
: 'RUN: at line 2';     /Users/buildslave/jenkins/workspace/clang-stage1-RA/clang-build/./bin/clang  --driver-mode=g++ -O2 -gline-tables-only -fsanitize=address,fuzzer -I/Users/buildslave/jenkins/workspace/clang-stage1-RA/llvm-project/compiler-rt/lib/fuzzer -arch x86_64 -stdlib=libc++ -mmacosx-version-min=10.9 -isysroot /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.15.sdk /Users/buildslave/jenkins/workspace/clang-stage1-RA/llvm-project/compiler-rt/test/fuzzer/LoadTest.cpp -fsanitize-coverage=trace-gep -o /Users/buildslave/jenkins/workspace/clang-stage1-RA/clang-build/tools/clang/runtime/compiler-rt-bins/test/fuzzer/X86_64DefaultDarwinConfig/Output/value-profile-load.test.tmp-LoadTest
: 'RUN: at line 3';   not  /Users/buildslave/jenkins/workspace/clang-stage1-RA/clang-build/tools/clang/runtime/compiler-rt-bins/test/fuzzer/X86_64DefaultDarwinConfig/Output/value-profile-load.test.tmp-LoadTest -seed=2 -use_cmp=0 -use_value_profile=1 -runs=20000000 2>&1 | FileCheck /Users/buildslave/jenkins/workspace/clang-stage1-RA/llvm-project/compiler-rt/test/fuzzer/value-profile-load.test
--
Exit Code: 1

Aug 19 2020, 10:36 AM · Restricted Project

Aug 11 2020

azharudd requested review of D85773: [Driver][XRay][test] Update the macOS support check.

Aug 11 2020, 12:58 PM · Restricted Project

Aug 6 2020

azharudd requested changes to D85173: [zorg] build.py: Allow lit test timeout value to be configurable.

Aug 6 2020, 10:46 AM

Aug 3 2020

azharudd added a reverting change for rZORG17e1c3a1e4ac: Temporarily disable email notification for test-suite pipeline jobs: rZORG334aee1e5a0e: Revert "Temporarily disable email notification for test-suite pipeline jobs".

Aug 3 2020, 5:19 PM

azharudd committed rZORG334aee1e5a0e: Revert "Temporarily disable email notification for test-suite pipeline jobs" (authored by azharudd).

Revert "Temporarily disable email notification for test-suite pipeline jobs"

Aug 3 2020, 5:19 PM

azharudd updated subscribers of D84703: [clang codegen][AArch64] Use llvm.aarch64.neon.fcvtzs/u where it's necessary.

Hey Eli,

Aug 3 2020, 11:01 AM · Restricted Project

Jul 28 2020

azharudd added a comment to D84700: [ThinLTO] [test] cache.ll: Prevent Spotlight indexing of the output dir.

In D84700#2178300, @jhenderson wrote:

No issues from my point of view, aside from the suggested comment change.

Jul 28 2020, 9:23 PM · Restricted Project

Jul 27 2020

azharudd added a comment to D84700: [ThinLTO] [test] cache.ll: Prevent Spotlight indexing of the output dir.

Since it is just a change in the directory name, I think this would still work fine on other platforms.

Jul 27 2020, 1:56 PM · Restricted Project

azharudd added a reviewer for D84700: [ThinLTO] [test] cache.ll: Prevent Spotlight indexing of the output dir: steven_wu.

Jul 27 2020, 1:49 PM · Restricted Project

Herald added a project to D84700: [ThinLTO] [test] cache.ll: Prevent Spotlight indexing of the output dir: Restricted Project.

Jul 27 2020, 1:28 PM · Restricted Project

May 27 2020

azharudd added a comment to D80474: tsan: fix false positives in AcquireGlobal.

Looks like the test added here is failing on Darwin. Can you please look into it.

May 27 2020, 6:34 PM

Dec 12 2019

azharudd committed rGfb31b13f352a: [AST] Fix test for Swift clang (authored by lanza).

[AST] Fix test for Swift clang

Dec 12 2019, 2:05 PM

Aug 15 2019

azharudd added a comment to D66296: [BUNDLER]Improve the test, NFC..

Looks like this is failing on Darwin:

Aug 15 2019, 11:57 AM · Restricted Project, Restricted Project

Jul 17 2019

azharudd committed rL366297: Revert "[zorg] Add a new build script for use with monorepo".

Revert "[zorg] Add a new build script for use with monorepo"

Jul 17 2019, 12:51 AM

Jul 8 2019

azharudd committed rL365440: [zorg] Add a new build script for use with monorepo.

[zorg] Add a new build script for use with monorepo

Jul 8 2019, 10:23 PM

azharudd closed D64028: [zorg] Add a new build script for use with monorepo.

Jul 8 2019, 10:23 PM · Restricted Project

Jul 1 2019

azharudd removed a reviewer for D64028: [zorg] Add a new build script for use with monorepo: azhar.

Jul 1 2019, 12:27 PM · Restricted Project

azharudd commandeered D64028: [zorg] Add a new build script for use with monorepo.

Jul 1 2019, 12:27 PM · Restricted Project

Nov 1 2018

azharudd added a comment to D53912: [Headers] [MS] Add intrin0.h.

In D53912#1281584, @rnk wrote:

This sounds like it would defeat what I'm assuming is the intended purpose of intrin0.h, which is to reduce compile time. intrin.h is kind of enormous, and the compile time problems are well-documented. We should investigate what's up with intrin0.h and implement as many builtins as we need to support it.

Nov 1 2018, 4:30 PM

Oct 30 2018

azharudd created D53912: [Headers] [MS] Add intrin0.h.

Oct 30 2018, 4:50 PM

Oct 5 2018

azharudd added a comment to D51804: [CMake] Fix LTO option on Windows.

CMAKE_LINKER MATCHES "lld-link\.exe"

@aganea, can you please elaborate on the need for the backslash that you added?

Oct 5 2018, 6:17 PM

Sep 24 2018

azharudd added a reviewer for D52144: use __ARM_FP instead of __VFP_FP__: peter.smith.

Sep 24 2018, 12:01 PM

Sep 13 2018

azharudd committed rLLD342145: Add missing REQUIRES x86 to test/COFF/icf-safe.s.

Add missing REQUIRES x86 to test/COFF/icf-safe.s

Sep 13 2018, 8:43 AM

azharudd committed rL342145: Add missing REQUIRES x86 to test/COFF/icf-safe.s.

Add missing REQUIRES x86 to test/COFF/icf-safe.s

Sep 13 2018, 8:43 AM

azharudd closed D51764: Add missing REQUIRES x86 to test/COFF/icf-safe.s.

Sep 13 2018, 8:43 AM

azharudd committed rLLD342144: Fix typo in error message. NFC..

Fix typo in error message. NFC.

Sep 13 2018, 8:40 AM

azharudd added a comment to D51764: Add missing REQUIRES x86 to test/COFF/icf-safe.s.

Thanks for the review @ruiu.

Sep 13 2018, 8:39 AM

azharudd committed rL342144: Fix typo in error message. NFC..

Fix typo in error message. NFC.

Sep 13 2018, 8:38 AM

azharudd closed D51767: Fix typo in error message. NFC..

Sep 13 2018, 8:38 AM

azharudd added a comment to D51767: Fix typo in error message. NFC..

Thanks for the review @ruiu.

Sep 13 2018, 8:37 AM

Sep 6 2018

azharudd created D51767: Fix typo in error message. NFC..

Sep 6 2018, 5:51 PM

azharudd created D51764: Add missing REQUIRES x86 to test/COFF/icf-safe.s.

Sep 6 2018, 5:08 PM

Aug 8 2018

azharudd added a reviewer for D50431: generalize SKS key server in debian8 Dockerfile: ilya-biryukov.

Adding Ilya who is the original author of that file.

Aug 8 2018, 4:06 PM

Jul 30 2018

azharudd committed rCRT338284: [cmake] [ARM] Exclude any VFP builtins if VFP is not supported.

[cmake] [ARM] Exclude any VFP builtins if VFP is not supported

Jul 30 2018, 11:20 AM

azharudd committed rL338284: [cmake] [ARM] Exclude any VFP builtins if VFP is not supported.

[cmake] [ARM] Exclude any VFP builtins if VFP is not supported

Jul 30 2018, 11:19 AM

azharudd closed D47217: [cmake] [ARM] Exclude any VFP builtins if VFP is not supported.

Jul 30 2018, 11:19 AM

azharudd added a comment to D47217: [cmake] [ARM] Exclude any VFP builtins if VFP is not supported.

@peter.smith Thanks for the review.

Jul 30 2018, 11:17 AM

Jul 27 2018

azharudd added a comment to D47217: [cmake] [ARM] Exclude any VFP builtins if VFP is not supported.

Ping for review please.

Jul 27 2018, 9:53 AM

Jul 24 2018

azharudd committed rL337842: [docker] Fix LLVM_EXTERNAL_PROJECTS cmake variable value.

[docker] Fix LLVM_EXTERNAL_PROJECTS cmake variable value

Jul 24 2018, 11:34 AM

azharudd closed D49712: [docker] Fix LLVM_EXTERNAL_PROJECTS cmake variable value.

Jul 24 2018, 11:34 AM

Jul 23 2018

azharudd created D49712: [docker] Fix LLVM_EXTERNAL_PROJECTS cmake variable value.

Jul 23 2018, 6:35 PM

azharudd updated the diff for D47217: [cmake] [ARM] Exclude any VFP builtins if VFP is not supported.

Updated to check if VFP is enabled for armhf too.

Jul 23 2018, 6:01 PM

azharudd added a comment to D47217: [cmake] [ARM] Exclude any VFP builtins if VFP is not supported.

In D47217#1165279, @peter.smith wrote:
Can you elaborate on why the armhf buildbot failed and why you have excluded armhf from the MATCHES expression above. A simple test of a source file containing
__VFP_FP__
clang --target=arm-linux-gnueabihf -E vfp.c gives me
# 1 "vfp.c"
# 1 "<built-in>" 1
# 1 "<built-in>" 3
# 361 "<built-in>" 3
# 1 "<command line>" 1
# 1 "<built-in>" 2
# 1 "vfp.c" 2
1
Unless I'm missing something a target of arm-linux-gnueabihf would define the macro unless the fpu were disabled with -mfpu=none so it should come through the test with the right answer.

Jul 23 2018, 6:01 PM

azharudd added a comment to D49397: Pass CMAKE_C_FLAGS to the custom libc++ build too.

In D49397#1166067, @phosek wrote:

What about C++ flags? libc++ is a C++ library so those are likely going to be important as well. Rather than lumping both C and C++ flags together into a single variable, we should probably consider adding two new keyword arguments to add_custom_libcxx, CMAKE_C_FLAGS and CMAKE_CXX_FLAGS, and use those to pass C and C++ flags separately to libc++ build.

Jul 23 2018, 12:26 PM

Jul 16 2018

azharudd created D49397: Pass CMAKE_C_FLAGS to the custom libc++ build too.

Jul 16 2018, 12:23 PM

azharudd requested review of D47217: [cmake] [ARM] Exclude any VFP builtins if VFP is not supported.

Jul 16 2018, 12:03 PM

azharudd updated the diff for D47217: [cmake] [ARM] Exclude any VFP builtins if VFP is not supported.

I had to revert this last time as it was causing the armhf buildbot to fail. This should take care of that.

Jul 16 2018, 12:03 PM

azharudd retitled D47217: [cmake] [ARM] Exclude any VFP builtins if VFP is not supported from [cmake] [ARM] Check if VFP is supported before including any VFP builtins to [cmake] [ARM] Exclude any VFP builtins if VFP is not supported.

Jul 16 2018, 12:00 PM

azharudd reopened D47217: [cmake] [ARM] Exclude any VFP builtins if VFP is not supported.

Jul 16 2018, 11:57 AM

Jul 15 2018

azharudd committed rOMP337133: [cmake] Fix libomptarget/test/CMakeLists.txt.

[cmake] Fix libomptarget/test/CMakeLists.txt

Jul 15 2018, 10:38 AM

azharudd committed rL337133: [cmake] Fix libomptarget/test/CMakeLists.txt.

[cmake] Fix libomptarget/test/CMakeLists.txt

Jul 15 2018, 10:34 AM

azharudd closed D47221: [cmake] Fix libomptarget/test/CMakeLists.txt.

Jul 15 2018, 10:34 AM

May 31 2018

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

In D47224#1112505, @Hahnfeld wrote:

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.

May 31 2018, 3:09 PM