r345446 - Revert r345170 [along with its llvm counterpart r345169] as it makes Halide builds timeout.

2018-10-27 Thread Alina Sbirlea via cfe-commits
Author: asbirlea Date: Fri Oct 26 21:51:09 2018 New Revision: 345446 URL: http://llvm.org/viewvc/llvm-project?rev=345446&view=rev Log: Revert r345170 [along with its llvm counterpart r345169] as it makes Halide builds timeout. Modified: cfe/trunk/lib/Driver/ToolChains/Hexagon.cpp cfe/tru

Re: [PATCH] D55802: Change CGObjC to use objc intrinsics instead of runtime methods

2019-11-02 Thread Alina Sbirlea via cfe-commits
Following up on this, AFAICT this is working as intended on the LLVM side. The different decision is made in GlobalsAA + MemoryDependencyAnalysis. IIUC, the logic there is along the lines of: if I have a global G that's "internal" ("static" in C), and an intrinsic method M, then M cannot read from

Re: [PATCH] D55802: Change CGObjC to use objc intrinsics instead of runtime methods

2019-09-11 Thread Alina Sbirlea via cfe-commits
Hi David, Does this still reproduce? I'm seeing the load after the call to autoreleasePoolPop at ToT, but perhaps I'm not using the proper flags? I only checked out the github repo and did "clang -fobjc-runtime=gnustep-2.0 -O3 -emit-llvm -S libobjc2/Test/AssociatedObject.m" with a ToT clang. Than

[clang] e22de4e - [DominatorTree] Simplify ChildrenGetter.

2020-07-28 Thread Alina Sbirlea via cfe-commits
Author: Alina Sbirlea Date: 2020-07-28T15:44:20-07:00 New Revision: e22de4e46d1dd1aacc3a7060d24bcbe89908ba6c URL: https://github.com/llvm/llvm-project/commit/e22de4e46d1dd1aacc3a7060d24bcbe89908ba6c DIFF: https://github.com/llvm/llvm-project/commit/e22de4e46d1dd1aacc3a7060d24bcbe89908ba6c.diff

[clang] bd541b2 - [NewPassManager] Add assertions when getting statefull cached analysis.

2020-05-13 Thread Alina Sbirlea via cfe-commits
Author: Alina Sbirlea Date: 2020-05-13T12:38:38-07:00 New Revision: bd541b217f4d750391677144ccaa586874236f38 URL: https://github.com/llvm/llvm-project/commit/bd541b217f4d750391677144ccaa586874236f38 DIFF: https://github.com/llvm/llvm-project/commit/bd541b217f4d750391677144ccaa586874236f38.diff

[clang] a903749 - [DomTree] Replace ChildrenGetter with GraphTraits over GraphDiff.

2020-04-09 Thread Alina Sbirlea via cfe-commits
Author: Alina Sbirlea Date: 2020-04-09T18:08:39-07:00 New Revision: a90374988e4eb8c50d91e11f4e61cdbd5debb235 URL: https://github.com/llvm/llvm-project/commit/a90374988e4eb8c50d91e11f4e61cdbd5debb235 DIFF: https://github.com/llvm/llvm-project/commit/a90374988e4eb8c50d91e11f4e61cdbd5debb235.diff

Re: [PATCH] D77341: [DomTree] Replace ChildrenGetter with GraphTraits over GraphDiff.

2020-04-09 Thread Alina Sbirlea via cfe-commits
I'm hoping rG5da1671bf823 fixes them. Please let me know if not and I will revert. Alina On Thu, Apr 9, 2020 at 7:03 PM Stella Stamenova via Phabricator < revi...@reviews.llvm.org> wrote: > stella.stamenova added a comment. > > It looks like this broke the windows lldb buildbot: > > lab.llvm.org

Re: [PATCH] D55802: Change CGObjC to use objc intrinsics instead of runtime methods

2019-10-18 Thread Alina Sbirlea via cfe-commits
I only got a chance to look more into this now. I can reproduce it with re-inserting the "static". The miscompile is not related to MemorySSA, i.e. disabling all uses of MemorySSA doesn't help. It appears to be a GVN bug, but I haven't tracked it further than this. To repro, see attached .ll file

[clang] 90bdb03 - Update clang test.

2020-01-17 Thread Alina Sbirlea via cfe-commits
Author: Alina Sbirlea Date: 2020-01-17T11:08:59-08:00 New Revision: 90bdb037276be43df260af004f620b21645d2617 URL: https://github.com/llvm/llvm-project/commit/90bdb037276be43df260af004f620b21645d2617 DIFF: https://github.com/llvm/llvm-project/commit/90bdb037276be43df260af004f620b21645d2617.diff

r358765 - Update to use PipelineTuningOptions. Corresponds to llvm change: D59723.

2019-04-19 Thread Alina Sbirlea via cfe-commits
Author: asbirlea Date: Fri Apr 19 09:32:08 2019 New Revision: 358765 URL: http://llvm.org/viewvc/llvm-project?rev=358765&view=rev Log: Update to use PipelineTuningOptions. Corresponds to llvm change: D59723. Modified: cfe/trunk/lib/CodeGen/BackendUtil.cpp Modified: cfe/trunk/lib/CodeGen/Back

r361534 - [NewPassManager] Add tuning option: SLPVectorization [clang-change]

2019-05-23 Thread Alina Sbirlea via cfe-commits
Author: asbirlea Date: Thu May 23 11:51:02 2019 New Revision: 361534 URL: http://llvm.org/viewvc/llvm-project?rev=361534&view=rev Log: [NewPassManager] Add tuning option: SLPVectorization [clang-change] Summary: NewPassManager is not using CodeGenOpts values before this patch. [to be coupled with

r361542 - Update breaking test.

2019-05-23 Thread Alina Sbirlea via cfe-commits
Author: asbirlea Date: Thu May 23 12:51:16 2019 New Revision: 361542 URL: http://llvm.org/viewvc/llvm-project?rev=361542&view=rev Log: Update breaking test. Modified: cfe/trunk/test/CodeGen/loop-vectorize.c Modified: cfe/trunk/test/CodeGen/loop-vectorize.c URL: http://llvm.org/viewvc/llvm-p

r361562 - Use clang_cc1 instead of clang in CodeGen test.

2019-05-23 Thread Alina Sbirlea via cfe-commits
Author: asbirlea Date: Thu May 23 15:07:37 2019 New Revision: 361562 URL: http://llvm.org/viewvc/llvm-project?rev=361562&view=rev Log: Use clang_cc1 instead of clang in CodeGen test. Modified: cfe/trunk/test/CodeGen/loop-vectorize.c Modified: cfe/trunk/test/CodeGen/loop-vectorize.c URL: htt

r361653 - [NewPassManager] Add tuning option: LoopUnrolling [clang-change]

2019-05-24 Thread Alina Sbirlea via cfe-commits
Author: asbirlea Date: Fri May 24 10:40:52 2019 New Revision: 361653 URL: http://llvm.org/viewvc/llvm-project?rev=361653&view=rev Log: [NewPassManager] Add tuning option: LoopUnrolling [clang-change] Summary: Use CodeGenOpts's setting for loop unrolling. [to be coupled with D61618] Reviewers: ch

r361674 - Mark tests as x86.

2019-05-24 Thread Alina Sbirlea via cfe-commits
Author: asbirlea Date: Fri May 24 14:49:27 2019 New Revision: 361674 URL: http://llvm.org/viewvc/llvm-project?rev=361674&view=rev Log: Mark tests as x86. Modified: cfe/trunk/test/CodeGen/loop-unroll.c cfe/trunk/test/CodeGen/loop-vectorize.c Modified: cfe/trunk/test/CodeGen/loop-unroll.c

Re: r361562 - Use clang_cc1 instead of clang in CodeGen test.

2019-05-24 Thread Alina Sbirlea via cfe-commits
<{{[0-9]+}} x double> > ^ > :9:25: note: scanning from here > define void @for_test() local_unnamed_addr #0 { > ^ > :11:5: note: possible intended match here > %A = alloca [1000 x double], align 16 > ^ > > -- > > Can y

r359616 - Set LoopInterleaved in the PassManagerBuilder.

2019-04-30 Thread Alina Sbirlea via cfe-commits
Author: asbirlea Date: Tue Apr 30 14:29:23 2019 New Revision: 359616 URL: http://llvm.org/viewvc/llvm-project?rev=359616&view=rev Log: Set LoopInterleaved in the PassManagerBuilder. Summary: Corresponds to D61030. Subscribers: jlebar, cfe-commits, llvm-commits Tags: #clang Differential Revisio

[clang] 577e0ef - [clang][AST] Silence unused-value warnings in unittest DeclPrinterTest

2024-03-26 Thread Alina Sbirlea via cfe-commits
Author: Alina Sbirlea Date: 2024-03-26T23:24:02-07:00 New Revision: 577e0ef94fb0b4ba9f97a6f58a1961f7ba247d21 URL: https://github.com/llvm/llvm-project/commit/577e0ef94fb0b4ba9f97a6f58a1961f7ba247d21 DIFF: https://github.com/llvm/llvm-project/commit/577e0ef94fb0b4ba9f97a6f58a1961f7ba247d21.diff