Pierre added a comment.

The wrong patch was uploaded. Sorry for this.



================
Comment at: clang/utils/TableGen/ClangOpenCLBuiltinEmitter.cpp:107
+  //        <<double, double>, 35>.
+  // TODO: Check the other types of vector available for performance purpose
+  std::vector<std::pair<std::vector<Record *>, unsigned>> SignatureSet;
----------------
Anastasia wrote:
> Is this TODO still relevant?
This is still relevant, but not for this patch I think.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D60763/new/

https://reviews.llvm.org/D60763



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to