[Clang][RISCV] Change TARGET_BUILTIN to require zve32x for vector instruction

[eopXD]

Summary

According to v-spec v1.0, zve-32x is the new minimum extension to include
to have vector instructions.

[eopXD]

Update code.

[eopXD]

Rebase.

[eopXD]

Follow clang-format.

[craig.topper]

The title of this patch isn't descriptive of the changes in this patch. Based on the title I would only expect changes to RISCVEmitter.cpp. Can you clarify the title?

[craig.topper]

clang/test/CodeGen/RISCV/rvv-intrinsics-overloaded/vaadd.c

3 ↗

(On Diff #383983)

Why do we need to add zve32x and zve64x to the command line? Doesn't zve64x imply zve32x?

[eopXD]

clang/test/CodeGen/RISCV/rvv-intrinsics-overloaded/vaadd.c

3 ↗

(On Diff #383983)

Just added D113336 to resolve this. I will rebase this patch and change test cases.

[eopXD]

Rebase.

[eopXD]

Update code.

[eopXD]

clang/test/CodeGen/RISCV/rvv-intrinsics-overloaded/vaadd.c

3 ↗

(On Diff #383983)

Rebased.

[eopXD]

Rebase.

[eopXD]

Rebase.

[frasercrmck]

LGTM but I don't have full scope on the builtins side of things. I'd also add a missing word to the title: "Change TARGET_BUILTIN to require ..."

[eopXD]

Apply @frasercrmck 's suggested title, thanks!

[kito-cheng]

LGTM

clang/utils/TableGen/RISCVVEmitter.cpp

void RVVEmitter::createBuiltins(raw_ostream &OS) {
   std::vector<std::unique_ptr<RVVIntrinsic>> Defs;
   createRVVIntrinsics(Defs);
 
   // Map to keep track of which builtin names have already been emitted.
   StringMap<RVVIntrinsic *> BuiltinMap;
 
   OS << "#if defined(TARGET_BUILTIN) && !defined(RISCVV_BUILTIN)\n";
   OS << "#define RISCVV_BUILTIN(ID, TYPE, ATTRS) TARGET_BUILTIN(ID, TYPE, "
-        "ATTRS, \"experimental-v\")\n";
+        "ATTRS, \"experimental-zve32x\")\n";
   OS << "#endif\n";
   for (auto &Def : Defs) {
     auto P =
         BuiltinMap.insert(std::make_pair(Def->getBuiltinName(), Def.get()));
     if (!P.second) {
       // Verify that this would have produced the same builtin definition.
       if (P.first->second->hasAutoDef() != Def->hasAutoDef()) {
         PrintFatalError("Builtin with same name has different hasAutoDef");