[PATCH] D88141: Update Kaleidoscope tutorial and remove useless header
    Eugene Zelenko via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Wed Sep 23 06:39:55 PDT 2020
    
    
  
Eugene.Zelenko added a comment.
Looks OK for me, but `utility` should be also included because of `std::move`.
In future I could recommend to use Include What You Use <https://github.com/include-what-you-use/include-what-you-use>. Will be good idea to do similar clean-ups for other examples.
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D88141/new/
https://reviews.llvm.org/D88141
    
    
More information about the llvm-commits
mailing list