[PATCH] D54398: [IPSCCP] Delete two forward declarations
Florian Hahn via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Nov 11 14:36:58 PST 2018
fhahn accepted this revision.
fhahn added a comment.
This revision is now accepted and ready to land.
Thanks for adding the include, I just saw the commit after I submitted the forward declaration.
If the forward declaration is sufficient (unfortunately the compiler I was using was clever enough to not require it...), I think it would be slightly better to remove the include. Both approaches LGTM, thank you very much!
Repository:
rL LLVM
https://reviews.llvm.org/D54398
More information about the llvm-commits
mailing list