You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2022/08/30 12:34:20 UTC

[GitHub] [arrow] js8544 commented on pull request #14004: ARROW-17567: [C++] Avoid internal compiler error with gcc 7 and c++17

js8544 commented on PR #14004:
URL: https://github.com/apache/arrow/pull/14004#issuecomment-1231605178

   > > Capturing the const ints by value fixs this issue and is a performance improvement.
   > 
   > I don't really believe the performance improvement claim (do you have any benchmark numbers?), but it's fine as a workaround.
   
   @pitrou You are right. I should have said "this workaround won't harm anything" rather than calling it an improvement.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org