[llvm] Remove out-of-date fixme from SmallVectorMemoryBuffer
authorJan Svoboda <jan_svoboda@apple.com>
Wed, 8 Dec 2021 10:43:43 +0000 (11:43 +0100)
committerJan Svoboda <jan_svoboda@apple.com>
Wed, 8 Dec 2021 10:53:50 +0000 (11:53 +0100)
This fixme first appeared in the codebase with the introduction of `ObjectMemoryBuffer` in rG93de2a12a36feaf1e6d8ff28c76db9b6cda4e844, but the constructor appears to never have been templated.

Reviewed By: dexonsmith

Differential Revision: https://reviews.llvm.org/D115044

llvm/include/llvm/Support/SmallVectorMemoryBuffer.h

index 9aa4e9a..af748cf 100644 (file)
@@ -30,12 +30,6 @@ class SmallVectorMemoryBuffer : public MemoryBuffer {
 public:
   /// Construct an SmallVectorMemoryBuffer from the given SmallVector
   /// r-value.
-  ///
-  /// FIXME: It'd be nice for this to be a non-templated constructor taking a
-  /// SmallVectorImpl here instead of a templated one taking a SmallVector<N>,
-  /// but SmallVector's move-construction/assignment currently only take
-  /// SmallVectors. If/when that is fixed we can simplify this constructor and
-  /// the following one.
   SmallVectorMemoryBuffer(SmallVectorImpl<char> &&SV)
       : SV(std::move(SV)), BufferName("<in-memory object>") {
     init(this->SV.begin(), this->SV.end(), false);