Page MenuHomePhabricator

jeroen.dobbelaere (Jeroen Dobbelaere)
User

Projects

User does not belong to any projects.

User Details

User Since
Mar 4 2015, 5:47 AM (236 w, 6 d)

Recent Activity

Wed, Sep 4

jeroen.dobbelaere added a comment to D67166: Win: handle \\?\UNC\ prefix in realPathFromHandle (PR43204).

Hi Hans, I am not too familiar with all windows path naming variants. Did you really observe a '\\?UNC\xxxx' path ?

Wed, Sep 4, 8:55 AM · Restricted Project

Jun 12 2019

Herald added a project to D39455: [CodeGen] Add initial support for union members in TBAA: Restricted Project.
Jun 12 2019, 8:33 AM · Restricted Project, Restricted Project

Jun 4 2019

Herald added a project to D41562: [CodeGen] Generate TBAA info on passing arguments and returning values: Restricted Project.
Jun 4 2019, 8:58 AM · Restricted Project, Restricted Project

Sep 26 2018

jeroen.dobbelaere added a comment to D9375: An llvm.noalias intrinsic.

Hi Hal,

Sep 26 2018, 10:42 AM

Sep 17 2018

jeroen.dobbelaere added inline comments to D9400: llvm.noalias - Use noalias intrinsics when inlining (and update them when cloning metadata).
Sep 17 2018, 9:11 AM

Sep 16 2018

jeroen.dobbelaere added a comment to D9400: llvm.noalias - Use noalias intrinsics when inlining (and update them when cloning metadata).

Compare following testcases (which should have an equivalent result):

Sep 16 2018, 10:08 AM

Aug 7 2018

jeroen.dobbelaere added a comment to D49467: Fix inconsistency with/without debug information (-g).

could one of you commit this change ?

Aug 7 2018, 4:24 AM

Jul 27 2018

jeroen.dobbelaere added a comment to D49467: Fix inconsistency with/without debug information (-g).

! In D49467#1177034, @probinson wrote:
Oh nice! I wasn't aware of that trick.

You just need to remember to take a copy of the 'comments' in your file and to copy those manually. Otherwise, the test driver is gone (as I noticed the hard way).

Jul 27 2018, 3:11 AM
jeroen.dobbelaere updated the diff for D49467: Fix inconsistency with/without debug information (-g).

Reduced function attributes.

Jul 27 2018, 3:09 AM

Jul 26 2018

jeroen.dobbelaere updated the diff for D49467: Fix inconsistency with/without debug information (-g).

Reassembled to reset the debug id's

Jul 26 2018, 8:50 AM

Jul 24 2018

jeroen.dobbelaere updated the diff for D49467: Fix inconsistency with/without debug information (-g).

Found one extra line of debug info that could be removed.

Jul 24 2018, 1:46 AM
jeroen.dobbelaere updated the diff for D49467: Fix inconsistency with/without debug information (-g).

Reduced the debug information.

Jul 24 2018, 1:43 AM

Jul 20 2018

jeroen.dobbelaere updated the diff for D49467: Fix inconsistency with/without debug information (-g).

Updated.

Jul 20 2018, 8:56 AM
jeroen.dobbelaere updated the diff for D49467: Fix inconsistency with/without debug information (-g).
  • Updated existing (related) testcase to actually test something
  • Added new testcase showing the problem
Jul 20 2018, 3:44 AM

Jul 19 2018

jeroen.dobbelaere added a comment to D49467: Fix inconsistency with/without debug information (-g).

Hi Jeroen, any chance you could add a small test to ensure we'd notice if this regresses in the future?

Jul 19 2018, 9:19 AM

Jul 18 2018

jeroen.dobbelaere created D49467: Fix inconsistency with/without debug information (-g).
Jul 18 2018, 2:58 AM

Jun 21 2018

jeroen.dobbelaere added a comment to D28896: [FileCheck] Add directive for checking for blank lines.

How about using an empty 'CHECK-NEXT:' statement for this ?

Jun 21 2018, 3:09 AM

Jun 5 2018

jeroen.dobbelaere added inline comments to D47770: [MVT][SVE] Add EVT strings and Type mapping.
Jun 5 2018, 5:23 AM · Restricted Project

Aug 7 2017

jeroen.dobbelaere added a comment to D9401: llvm.noalias - The AA implementaton.

Hi Hal, I found an issue with this part when analyzing two pointers that are 'derived from each other', but are used in different loops.
Following code would go wrong:

Aug 7 2017, 6:20 AM

Oct 17 2016

jeroen.dobbelaere added inline comments to D9401: llvm.noalias - The AA implementaton.
Oct 17 2016, 7:55 AM
jeroen.dobbelaere added a comment to D9386: llvm.noalias - CaptureTracking needs to look through them.

Maybe 'passthrough' should return a boolean, indicating that we need to stop or not. (see code comment)
The places where it is called could become something like: 'if (passthrough()) return;

Oct 17 2016, 3:10 AM

Mar 17 2015

jeroen.dobbelaere abandoned D8056: Fix for bug 21725: wrong results with union and strict-aliasing.

Further investigation shows that llvm/lib/Analysis/TypeBasedAliasAnalysis.cpp is not ready to handle struct types where multiple members have the same (0) offset. The clang fix only makes sense once that issue is resolved.

Mar 17 2015, 7:37 AM

Mar 4 2015

jeroen.dobbelaere retitled D8056: Fix for bug 21725: wrong results with union and strict-aliasing from to Fix for bug 21725: wrong results with union and strict-aliasing.
Mar 4 2015, 6:01 AM