[clang] [AArch64] Implement reinterpret builtins for SVE vector tuples (PR #69598)

via cfe-commits cfe-commits at lists.llvm.org
Thu Oct 26 10:41:43 PDT 2023


================
@@ -954,13 +967,13 @@ std::string Intrinsic::replaceTemplatedArgs(std::string Name, TypeSpec TS,
     default:
       llvm_unreachable("Unknown predication specifier");
     case 'd':
-      T = SVEType(TS, 'd');
+      T = SVEType(TS, 'd', 1);
----------------
CarolineConcatto wrote:

I think it is fine, but in many classes the SVEType is being initialised with std_string.  So StringRef and std::string are being used as interchangeable.
https://discourse.llvm.org/t/std-string-vs-llvm-stringref/65873/3

https://github.com/llvm/llvm-project/pull/69598


More information about the cfe-commits mailing list