Page MenuHomePhabricator

cbergstrom (Christopher Bergström)
UserEmail Not Verified


User does not belong to any projects.

User Details

User Since
Jul 15 2013, 12:44 AM (467 w, 2 d)

Recent Activity

Feb 11 2016

cbergstrom added a comment to D14031: [OpenMP] Initial implementation of OpenMP offloading library - libomptarget..

Please take this comment as more a nitpick instead of having tested it

In general I see a lot of really dirty code - Why so many casts? What makes "long long" universally correct and portable.. are there standard types we can use to represent pointers?

Feb 11 2016, 12:07 PM

Oct 14 2015

cbergstrom added a comment to D13739: [libcxx] Make a linker script by default on most platforms..

‎And does this hardcode the abi and break cxxrt? It's not just me who uses that 

Oct 14 2015, 1:41 PM

Jul 20 2015

cbergstrom added a comment to D11301: Remove the __kmp_invoke_microtask() that relies on libffi.

Why is libffi a depdendency at all? What if it's not installed? What's
used on OSX and Windows? If it's performance impacting should a more
portable altenternative be explored?

Jul 20 2015, 5:59 AM

Jun 26 2015

cbergstrom added a comment to D10656: LLVM OpenMP CMake Overhaul.

One of the previous comments in this thread was that there is an
assumption that libomp will be installed to /usr/lib or some system
directory. I absolutely don't believe that should be done. The
-fopenmp or equivalent flag should add the correct (private) includes
(C/C++/Fortran module) and -L/-rpath lib location. This is really
compiler internal and private. LLVM was the odd duck by relying on the
gcc OMP lib before. If llvm had Fortran support - that wouldn't likely
have ever happened from the start.

Jun 26 2015, 2:22 AM

Jun 23 2015

cbergstrom added a comment to D10656: LLVM OpenMP CMake Overhaul.

Is this patch updated to reflect my review comments?

Jun 23 2015, 9:47 AM

Sep 21 2014

cbergstrom added a comment to D4002: [OPENMP] Codegen for threadprivate variables.

  Original Message  
From: Alexey Bataev
Sent: Monday, September 22, 2014 10:34
Reply To:
Subject: Re: [PATCH] [OPENMP] Codegen for threadprivate variables

Sep 21 2014, 11:26 PM