[PATCH] D70089: [VFABI] Add LLVM internal mangling for vector functions.

Sander de Smalen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 12 01:56:25 PST 2019


sdesmalen accepted this revision.
sdesmalen added a comment.
This revision is now accepted and ready to land.

Nice and simple patch, LGTM!



================
Comment at: llvm/include/llvm/Analysis/VectorUtils.h:50
   AVX512,       // x86 AVX512
-  Unknown       // Unknown ISA
+  LLVM_VF,      // LLVM internal ISA for functions that are not
+  // attached to an existing ABI via name mangling.
----------------
nit: maybe drop the `_VF` and use `LLVM` directly? (also for other uses in this file)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70089





More information about the llvm-commits mailing list