[PATCH] D19154: Remove some unneeded headers and replace some headers with forward class declarations

Eugene via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 15 11:55:16 PDT 2016


kevgs added a comment.

Thanks for your review. I have no commit rights.


================
Comment at: include/llvm/ADT/APFloat.h:29
@@ -28,1 +28,3 @@
+template <typename T> class SmallVectorImpl;
+
 /// Enum that represents what fraction of the LSB truncated bits of an fp number
----------------
joker.eph wrote:
> You can still include APFloat without including SmallVector right?
Yes, unless you're using SmallVector in your code.


http://reviews.llvm.org/D19154





More information about the llvm-commits mailing list