[PATCH] D66024: Name Demangling as specified in the Vector Function ABI

[Sumedh Arani via Phabricator via llvm-commits]

aranisumedh added inline comments.


================
Comment at: llvm/include/llvm/Analysis/SearchVectorFunctionSystem.h:29
+namespace llvm {
+
+/// Describes the type of Parameters
----------------
lebedev.ri wrote:
> All these are currently in llvm namespace, would it be better to narrow it and put them into `VFABI`?
Someone correct me if I am wrong. 

This module is intended to be extensible to other vectorization as well (as stated in the RFC). I am assuming that it can be extended to other ParameterKind as well. 

Whereas the name demangling for now is as stated in the vector function ABI and hence a part of the VFABI namespace.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D66024