[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