[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