Page MenuHomePhabricator

cbergstrom (Christopher Bergström)Email Not Verified
User

Projects

User does not belong to any projects.

User Details

User Since
Jul 15 2013, 12:44 AM (322 w, 5 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
cbergstrom updated subscribers of D14031: [OpenMP] Initial implementation of OpenMP offloading library - libomptarget..
Feb 11 2016, 11:52 AM

Oct 14 2015

cbergstrom added a comment to D13739: [libcxx] Make libc++.so 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
To: a.bataev@hotmail.com; dgregor@apple.com; hfinkel@anl.gov; cbergstrom@pathscale.com; rjmccall@gmail.com; fraggamuffin@gmail.com; ejstotzer@gmail.com; richard@metafoo.co.uk
Reply To: reviews+D4002+public+e0b72e4194bfb02b@reviews.llvm.org
Cc: cfe-commits@cs.uiuc.edu
Subject: Re: [PATCH] [OPENMP] Codegen for threadprivate variables

Sep 21 2014, 11:26 PM