Page MenuHomePhabricator

stevewan (Steven Wan)
User

Projects

User does not belong to any projects.

User Details

User Since
Sep 26 2019, 10:57 AM (46 w, 8 h)

A hardwired bot that knows only how to emit "LGTM".

Recent Activity

Jul 13 2020

stevewan committed rGa7e9c5a39fbc: [llvm-ar][test][AIX] Unsupport error-opening-directory.test on AIX (authored by stevewan).
[llvm-ar][test][AIX] Unsupport error-opening-directory.test on AIX
Jul 13 2020, 9:00 AM
stevewan closed D83579: [llvm-ar][test][AIX] Unsupport error-opening-directory.test on AIX.
Jul 13 2020, 9:00 AM · Restricted Project
stevewan accepted D83055: [clang][Driver] Fix tool path priority test failures.

This LGTM, but since I'm not most familiar with the type 1 failure in description, let's see if other reviewers have further comments.

Jul 13 2020, 8:46 AM · Restricted Project

Jul 10 2020

stevewan added a comment to D83579: [llvm-ar][test][AIX] Unsupport error-opening-directory.test on AIX.

FYI @sameerarora101, we on AIX are experiencing the exact same failure as what has been fixed in D82786 for FreeBSD. This patch adds AIX to the unsupported list as well.

Jul 10 2020, 11:38 AM · Restricted Project
stevewan added reviewers for D83579: [llvm-ar][test][AIX] Unsupport error-opening-directory.test on AIX: sameerarora101, daltenty, ShuhongL, hubert.reinterpretcast.
Jul 10 2020, 11:35 AM · Restricted Project
Herald added a project to D83579: [llvm-ar][test][AIX] Unsupport error-opening-directory.test on AIX: Restricted Project.
Jul 10 2020, 11:34 AM · Restricted Project
stevewan added inline comments to D83055: [clang][Driver] Fix tool path priority test failures.
Jul 10 2020, 11:19 AM · Restricted Project

Jul 9 2020

stevewan added inline comments to D83055: [clang][Driver] Fix tool path priority test failures.
Jul 9 2020, 4:14 PM · Restricted Project
stevewan closed D83342: [test-suite][AIX] Fix C++ benchmark compatibility issues on AIX.

Committed in https://github.com/llvm/llvm-test-suite/commit/f0caeccfb893344d73474e9d8f6c034ae7f451f9.

Jul 9 2020, 10:45 AM

Jul 8 2020

stevewan updated the summary of D83342: [test-suite][AIX] Fix C++ benchmark compatibility issues on AIX.
Jul 8 2020, 3:30 PM
stevewan added inline comments to D83342: [test-suite][AIX] Fix C++ benchmark compatibility issues on AIX.
Jul 8 2020, 3:17 PM
stevewan updated the diff for D83342: [test-suite][AIX] Fix C++ benchmark compatibility issues on AIX.

Take a different path in the application to avoid using 'timezone'.

Jul 8 2020, 3:16 PM
stevewan added inline comments to D83342: [test-suite][AIX] Fix C++ benchmark compatibility issues on AIX.
Jul 8 2020, 2:43 PM
stevewan added a comment to D79842: [clang][Driver] Correct tool search path priority.

Right, I see the issue.

The code that gets the default triple name (https://reviews.llvm.org/D13340?id=36227#inline-108606) looks up the one you have in cmake, not the actual default which you get in --version. We could "fix" this by doing so when we make the tool name as well, but this breaks whatever mips toolchain was using that. (their tools won't be mips-unknown-elf-<whatever>)

So yes it looks up powerpc64le-linux-gnu but shows powerpc64le-unknown-linux-gnu. Can't go back to using cmake's value because on Mac OS, cmake has x86_64-darwin, clang has x86_64-darwin<version>. Writing to both is a short term option so I will try that and fold it into https://reviews.llvm.org/D83055. (will add you on review once I update it)

(this whole default triple lookup should probably go but I'd really like to do that in its own commit)

Jul 8 2020, 2:06 PM · Restricted Project
stevewan added inline comments to D83342: [test-suite][AIX] Fix C++ benchmark compatibility issues on AIX.
Jul 8 2020, 11:35 AM
stevewan added inline comments to D83342: [test-suite][AIX] Fix C++ benchmark compatibility issues on AIX.
Jul 8 2020, 11:05 AM

Jul 7 2020

stevewan added reviewers for D83342: [test-suite][AIX] Fix C++ benchmark compatibility issues on AIX: hubert.reinterpretcast, daltenty, ShuhongL.
Jul 7 2020, 1:15 PM
stevewan created D83342: [test-suite][AIX] Fix C++ benchmark compatibility issues on AIX.
Jul 7 2020, 1:14 PM
stevewan added a comment to D79842: [clang][Driver] Correct tool search path priority.

Yes, this issue was hit with the reland applied. When given a <arch>-<sys>-<abi> format LLVM_DEFAULT_TARGET_TRIPLE, Clang would expand the target triple to <arch>-<vendor>-<sys>-<abi>, and therefore causes the name mismatch between what the driver searches for and what the test case creates as the dummy tool.

Jul 7 2020, 11:02 AM · Restricted Project

Jul 6 2020

stevewan added a comment to D82321: [AIX] [test-suite] Enable standard compatable mode on AIX.

This is failing on Darwin. Looks like for the C++ test too, the flag should be added only if it is AIX.

MultiSource/Benchmarks/DOE-ProxyApps-C++/miniFE/mytimer.cpp:116:19: error: variable has incomplete type 'struct timezone'

Refer to http://green.lab.llvm.org/green/job/test-suite-verify-machineinstrs-aarch64-O0-g/7783/consoleFull.

Jul 6 2020, 5:11 PM
stevewan added a comment to D82321: [AIX] [test-suite] Enable standard compatable mode on AIX.

This is failing on Darwin. Looks like for the C++ test too, the flag should be added only if it is AIX.

MultiSource/Benchmarks/DOE-ProxyApps-C++/miniFE/mytimer.cpp:116:19: error: variable has incomplete type 'struct timezone'

Refer to http://green.lab.llvm.org/green/job/test-suite-verify-machineinstrs-aarch64-O0-g/7783/consoleFull.

Jul 6 2020, 2:23 PM
stevewan added a comment to D82321: [AIX] [test-suite] Enable standard compatable mode on AIX.

This is failing on Darwin. Looks like for the C++ test too, the flag should be added only if it is AIX.

MultiSource/Benchmarks/DOE-ProxyApps-C++/miniFE/mytimer.cpp:116:19: error: variable has incomplete type 'struct timezone'

Refer to http://green.lab.llvm.org/green/job/test-suite-verify-machineinstrs-aarch64-O0-g/7783/consoleFull.

Jul 6 2020, 1:32 PM
stevewan added a comment to D79842: [clang][Driver] Correct tool search path priority.

The test was failing on Linux if I set LLVM_DEFAULT_TARGET_TRIPLE. For example if I set it to`powerpc64le-linux-gnu` clang actually uses "powerpc64le-unknown-linux-gnu".

Jul 6 2020, 12:31 PM · Restricted Project

Jul 3 2020

stevewan accepted D82677: [Clang] Handle AIX Include management in the driver.

LGTM with minor nit.

Jul 3 2020, 11:52 AM · Restricted Project

Jun 26 2020

stevewan added inline comments to D82677: [Clang] Handle AIX Include management in the driver.
Jun 26 2020, 3:18 PM · Restricted Project
stevewan added inline comments to D82677: [Clang] Handle AIX Include management in the driver.
Jun 26 2020, 1:41 PM · Restricted Project

Jun 24 2020

stevewan closed D82321: [AIX] [test-suite] Enable standard compatable mode on AIX.

Committed in https://github.com/llvm/llvm-test-suite/commit/000c7b8d78b12b5cc70ad4e69a4cdb1bab23f74d.

Jun 24 2020, 5:24 PM
stevewan updated the diff for D82321: [AIX] [test-suite] Enable standard compatable mode on AIX.

Add comment based on review.

Jun 24 2020, 11:21 AM

Jun 22 2020

stevewan retitled D82321: [AIX] [test-suite] Enable standard compatable mode on AIX from [AIX] Enable standard compatable mode by default on AIX to [AIX] [test-suite] Enable standard compatable mode on AIX.
Jun 22 2020, 12:21 PM
stevewan updated the summary of D82321: [AIX] [test-suite] Enable standard compatable mode on AIX.
Jun 22 2020, 12:21 PM
stevewan created D82321: [AIX] [test-suite] Enable standard compatable mode on AIX.
Jun 22 2020, 11:49 AM

Jun 19 2020

stevewan closed D82052: Remove unsupported case "AtomicOps" from AIX test list..

Committed in https://github.com/llvm/llvm-test-suite/commit/e4c53876d55063d459790218958c0fe03f0c6e1b.

Jun 19 2020, 12:31 PM

Jun 18 2020

stevewan updated the summary of D82052: Remove unsupported case "AtomicOps" from AIX test list..
Jun 18 2020, 10:18 AM

Jun 17 2020

stevewan created D82052: Remove unsupported case "AtomicOps" from AIX test list..
Jun 17 2020, 1:29 PM
stevewan added reviewers for D82052: Remove unsupported case "AtomicOps" from AIX test list.: hubert.reinterpretcast, daltenty.
Jun 17 2020, 1:29 PM
stevewan updated the summary of D82052: Remove unsupported case "AtomicOps" from AIX test list..
Jun 17 2020, 1:29 PM

Jun 5 2020

stevewan committed rG61cd264068ee: [PowerPC] Do not special case Darwin on PowerPC in target cpu handling (authored by stevewan).
[PowerPC] Do not special case Darwin on PowerPC in target cpu handling
Jun 5 2020, 5:22 PM
stevewan closed D81115: [PowerPC] Do not special case Darwin on PowerPC in target cpu handling.
Jun 5 2020, 5:22 PM · Restricted Project
stevewan added a comment to D81115: [PowerPC] Do not special case Darwin on PowerPC in target cpu handling.

The unit test failure (instrprof-gcov-multithread_fork.test) in pre merge checks is caused by another commit. See http://lab.llvm.org:8011/builders/llvm-avr-linux/builds/2205.

Jun 5 2020, 5:21 PM · Restricted Project
stevewan added a comment to D80966: [codeview] Put !heapallocsite on calls to operator new.

Looks like this has broken the build bot? (see http://lab.llvm.org:8011/builders/llvm-avr-linux/builds/2205)

Jun 5 2020, 5:21 PM · Restricted Project

Jun 4 2020

stevewan updated the diff for D81115: [PowerPC] Do not special case Darwin on PowerPC in target cpu handling.

Reduce the nested 'if' and reorder it to prefer the more likely case.

Jun 4 2020, 2:23 PM · Restricted Project
stevewan retitled D81115: [PowerPC] Do not special case Darwin on PowerPC in target cpu handling from [PowerPC] Do not check for non-Darwin in PowerPC target cpu handling to [PowerPC] Do not special case Darwin on PowerPC in target cpu handling.
Jun 4 2020, 2:23 PM · Restricted Project

Jun 3 2020

stevewan added inline comments to D80835: [AIX] Change the default target CPU to power4 for AIX on Power.
Jun 3 2020, 1:11 PM · Restricted Project
stevewan added reviewers for D81115: [PowerPC] Do not special case Darwin on PowerPC in target cpu handling: hubert.reinterpretcast, daltenty.
Jun 3 2020, 1:11 PM · Restricted Project
stevewan created D81115: [PowerPC] Do not special case Darwin on PowerPC in target cpu handling.
Jun 3 2020, 1:11 PM · Restricted Project
stevewan committed rGba4afe6f7a84: [AIX] Change the default target CPU to power4 for AIX on Power (authored by stevewan).
[AIX] Change the default target CPU to power4 for AIX on Power
Jun 3 2020, 11:00 AM
stevewan closed D80835: [AIX] Change the default target CPU to power4 for AIX on Power.
Jun 3 2020, 10:59 AM · Restricted Project

May 29 2020

stevewan added reviewers for D80835: [AIX] Change the default target CPU to power4 for AIX on Power: hubert.reinterpretcast, cebowleratibm, daltenty.
May 29 2020, 1:06 PM · Restricted Project
stevewan created D80835: [AIX] Change the default target CPU to power4 for AIX on Power.
May 29 2020, 1:06 PM · Restricted Project

May 26 2020

stevewan committed rGa924dac44f31: [NFC] Fix formatting for the 'aix-ld.c' test case. (authored by stevewan).
[NFC] Fix formatting for the 'aix-ld.c' test case.
May 26 2020, 3:18 PM
stevewan closed D80532: [NFC] Fix formatting for the 'aix-ld.c' test case..
May 26 2020, 3:18 PM · Restricted Project
stevewan updated the diff for D80532: [NFC] Fix formatting for the 'aix-ld.c' test case..

Add the two-space indent before continuations on a pipeine.

May 26 2020, 10:49 AM · Restricted Project

May 25 2020

stevewan updated the diff for D80532: [NFC] Fix formatting for the 'aix-ld.c' test case..

Pass 'clang-format' checking.

May 25 2020, 3:01 PM · Restricted Project
stevewan updated the diff for D80532: [NFC] Fix formatting for the 'aix-ld.c' test case..

Fix misalignment caused by tab.

May 25 2020, 1:56 PM · Restricted Project
stevewan added reviewers for D80532: [NFC] Fix formatting for the 'aix-ld.c' test case.: ZarkoCA, hubert.reinterpretcast, daltenty.
May 25 2020, 1:24 PM · Restricted Project
stevewan created D80532: [NFC] Fix formatting for the 'aix-ld.c' test case..
May 25 2020, 1:24 PM · Restricted Project
stevewan updated the summary of D80532: [NFC] Fix formatting for the 'aix-ld.c' test case..
May 25 2020, 1:24 PM · Restricted Project
stevewan committed rG5bf2409a4e4d: [AIX] Add '-bcdtors:all:0:s' to linker to gather static init functions (authored by stevewan).
[AIX] Add '-bcdtors:all:0:s' to linker to gather static init functions
May 25 2020, 11:15 AM
stevewan closed D80415: [AIX] Add '-bcdtors:all:0:s' to linker to gather static init functions.
May 25 2020, 11:15 AM · Restricted Project

May 22 2020

stevewan updated the diff for D80415: [AIX] Add '-bcdtors:all:0:s' to linker to gather static init functions.

Update check prefix based on comments.

May 22 2020, 5:10 PM · Restricted Project
stevewan added a comment to D80415: [AIX] Add '-bcdtors:all:0:s' to linker to gather static init functions.

I'm planning to post an NFC patch after this to fix the formatting for existing cases in 'aix-ld.c' so that everything is consistent.

May 22 2020, 4:06 PM · Restricted Project
stevewan updated the diff for D80415: [AIX] Add '-bcdtors:all:0:s' to linker to gather static init functions.

Incorporate the test case into 'aix-ld.c'.

May 22 2020, 4:06 PM · Restricted Project
stevewan added inline comments to D80415: [AIX] Add '-bcdtors:all:0:s' to linker to gather static init functions.
May 22 2020, 2:31 PM · Restricted Project
stevewan updated the diff for D80415: [AIX] Add '-bcdtors:all:0:s' to linker to gather static init functions.

Address furthur comments on the test case.

May 22 2020, 12:20 PM · Restricted Project
stevewan updated the diff for D80415: [AIX] Add '-bcdtors:all:0:s' to linker to gather static init functions.

Address comments and fix Windows compatibility issues with the test case.

May 22 2020, 10:10 AM · Restricted Project

May 21 2020

stevewan added reviewers for D80415: [AIX] Add '-bcdtors:all:0:s' to linker to gather static init functions: hubert.reinterpretcast, Xiangling_L, ZarkoCA, daltenty.
May 21 2020, 3:43 PM · Restricted Project
stevewan created D80415: [AIX] Add '-bcdtors:all:0:s' to linker to gather static init functions.
May 21 2020, 3:43 PM · Restricted Project

May 11 2020

stevewan committed rG44f7b4024982: Add vendor macro to "lld" (authored by stevewan).
Add vendor macro to "lld"
May 11 2020, 3:41 PM
stevewan closed D79026: Add vendor macro to "lld".
May 11 2020, 3:41 PM · Restricted Project

May 7 2020

stevewan planned changes to D77660: [test-suite] Workaround _ALL_SOURCE issue on AIX.
May 7 2020, 11:55 AM

Apr 29 2020

stevewan updated the diff for D79026: Add vendor macro to "lld".

Fix format with 'clang-format'.

Apr 29 2020, 11:48 AM · Restricted Project
stevewan updated the diff for D79026: Add vendor macro to "lld".

Fix format with 'clang-format'.

Apr 29 2020, 11:48 AM · Restricted Project

Apr 28 2020

stevewan created D79026: Add vendor macro to "lld".
Apr 28 2020, 10:44 AM · Restricted Project
stevewan added a reviewer for D79026: Add vendor macro to "lld": ruiu.
Apr 28 2020, 10:44 AM · Restricted Project
stevewan added reviewers for D79026: Add vendor macro to "lld": hubert.reinterpretcast, kbarton, cebowleratibm, rzurob.
Apr 28 2020, 10:44 AM · Restricted Project

Apr 7 2020

stevewan added reviewers for D77660: [test-suite] Workaround _ALL_SOURCE issue on AIX: hubert.reinterpretcast, daltenty.
Apr 7 2020, 9:45 AM
stevewan created D77660: [test-suite] Workaround _ALL_SOURCE issue on AIX.
Apr 7 2020, 9:45 AM

Mar 2 2020

stevewan added inline comments to D75305: [AIX] Handle LinkOnceODRLinkage and AppendingLinkage for static init gloabl arrays.
Mar 2 2020, 9:29 AM · Restricted Project

Feb 19 2020

stevewan added inline comments to D70972: [AIX] Make sure we use export lists for plugins.
Feb 19 2020, 8:45 AM · Restricted Project

Feb 14 2020

stevewan accepted D74256: [AIX] Improve 32/64-bit build configuration.

LGTM. Thanks.

Feb 14 2020, 9:08 AM · Restricted Project

Feb 13 2020

stevewan added inline comments to D74256: [AIX] Improve 32/64-bit build configuration.
Feb 13 2020, 1:11 PM · Restricted Project

Feb 12 2020

stevewan added inline comments to D74256: [AIX] Improve 32/64-bit build configuration.
Feb 12 2020, 12:06 PM · Restricted Project

Feb 11 2020

stevewan committed rG5e37fb1776de: [NFC] Delete extra white space in a test case. (authored by stevewan).
[NFC] Delete extra white space in a test case.
Feb 11 2020, 1:10 PM
stevewan added inline comments to D74256: [AIX] Improve 32/64-bit build configuration.
Feb 11 2020, 11:37 AM · Restricted Project
stevewan added inline comments to D74015: [AIX][Frontend] C++ ABI customizations for AIX boilerplate.
Feb 11 2020, 11:10 AM · Restricted Project, Restricted Project
stevewan added inline comments to D70972: [AIX] Make sure we use export lists for plugins.
Feb 11 2020, 8:48 AM · Restricted Project

Feb 10 2020

stevewan added inline comments to D74256: [AIX] Improve 32/64-bit build configuration.
Feb 10 2020, 8:38 AM · Restricted Project

Jan 30 2020

stevewan added a project to D72479: [PowerPC][AIX] Make PIC the default relocation model for AIX: Unknown Object (Project).
Jan 30 2020, 7:27 AM · Unknown Object (Project), Restricted Project

Jan 17 2020

stevewan added a comment to D72736: [AIX] Add improved interface for retrieving load module paths.

Provided that you have squashed all changes into the latest commit in your branch, git diff HEAD^ -U10000 should emit the context you need. Adjust the number of lines of context accordingly to fit the size of your files.

Jan 17 2020, 8:15 AM · Restricted Project, Restricted Project

Jan 16 2020

stevewan committed rGad83bed2411f: [NFC][PowerPC] Remove unnecessary link components. (authored by stevewan).
[NFC][PowerPC] Remove unnecessary link components.
Jan 16 2020, 6:24 PM
stevewan committed rG6c26d8968ad8: Add back more link components. (authored by stevewan).
Add back more link components.
Jan 16 2020, 5:20 PM
stevewan committed rG1b1022b42ade: Add back other PowerPC link components. (authored by stevewan).
Add back other PowerPC link components.
Jan 16 2020, 4:22 PM
stevewan added inline comments to D72479: [PowerPC][AIX] Make PIC the default relocation model for AIX.
Jan 16 2020, 1:26 PM · Unknown Object (Project), Restricted Project
stevewan committed rGfc4e43ad618b: [NFC] Remove unnecessary link components. (authored by stevewan).
[NFC] Remove unnecessary link components.
Jan 16 2020, 1:17 PM
stevewan added a comment to D72479: [PowerPC][AIX] Make PIC the default relocation model for AIX.

Also, this breaks tests on Windows: http://45.33.8.238/win/5946/step_11.txt

Thanks for noting, looking into it now.

Any luck? Bot's been broken for a while now. Maybe just remove the 1>/dev/null for now, or remove that test until things are figured out?

Jan 16 2020, 12:18 PM · Unknown Object (Project), Restricted Project
stevewan committed rGd0b02aecacc6: Address redirect issue on Windows. (authored by stevewan).
Address redirect issue on Windows.
Jan 16 2020, 12:09 PM
stevewan committed rGff1e3cad71e6: Don't run powerpc lit test case on other platforms. (authored by stevewan).
Don't run powerpc lit test case on other platforms.
Jan 16 2020, 11:39 AM
stevewan added a comment to D72479: [PowerPC][AIX] Make PIC the default relocation model for AIX.

Also, this breaks tests on Windows: http://45.33.8.238/win/5946/step_11.txt

Jan 16 2020, 10:41 AM · Unknown Object (Project), Restricted Project
stevewan committed rGbed7626f04f7: [PowerPC][AIX] Make PIC the default relocation model for AIX (authored by stevewan).
[PowerPC][AIX] Make PIC the default relocation model for AIX
Jan 16 2020, 10:13 AM
stevewan closed D72479: [PowerPC][AIX] Make PIC the default relocation model for AIX.
Jan 16 2020, 10:13 AM · Restricted Project, Restricted Project