[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