[PATCH] D117913: [Clang][RISCV] Guard vmulh, vsmul correctly

[Craig Topper via Phabricator via cfe-commits]

craig.topper added inline comments.


================
Comment at: clang/utils/TableGen/RISCVVEmitter.cpp:1321
   ListSeparator LS(" && ");
+  // Full multiplies for EEW=64 require V.
+  if (PredefinedMacros & RISCVPredefinedMacro::V)
----------------
It's not as useful here. Should put it on line 812 where the condition is checked and explain that a FullMultiply is MULH and SMUL.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D117913