craig.topper added inline comments.

================
Comment at: clang/test/CodeGen/RISCV/riscv-v-lifetime.cpp:3
+// REQUIRES: riscv-registered-target
+// RUN: %clang_cc1 -std=c++11 -triple riscv64 -target-feature +experimental-v \
+// RUN:   -emit-llvm -O1 -o - %s | FileCheck %s
----------------
Probably best to add -disable-llvm-passes so we don't run the optimizer.


================
Comment at: clang/test/CodeGen/RISCV/riscv-v-lifetime.cpp:8
+
+template <typename T>
+void Foo(T &&);
----------------
Are the templates needed? This was enough to fail for me

```
vint32m1_t Baz();                                                               
                                                                                
                                                                
                                                                                
                                                                                
                                                                
vint32m1_t Test() {                                                             
                                                                                
                                                                
  const vint32m1_t &a = Baz();                                                  
                                                                                
                                                                
  return a;                                                                     
                                                                                
                                                                
}
```

Or do the templates test additional code paths?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102822

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

Reply via email to