wenlei added a comment.

In D120610#3373042 <https://reviews.llvm.org/D120610#3373042>, @dblaikie wrote:

> Fixes in LLVM require tests in LLVM - probably taking the clang test and 
> compiling that to llvm IR (include the original C++ source in a comment in 
> the IR test case) and then testing it in LLVM instead of clang.
>
> Also looks like the test could be simplified a bit more:
>
>   void f1();
>   
>   template <typename T>
>   void f2() {
>     f1();
>   }
>   
>   void f3() {
>     f2<int>();
>   }

Done, thanks.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D120610

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

Reply via email to