[PATCH] D77594: [SveEmitter] Add support for _n form builtins

Sjoerd Meijer via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Apr 20 08:05:18 PDT 2020


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

Looks reasonable to me



================
Comment at: clang/utils/TableGen/SveEmitter.cpp:212
+  bool hasSplat() const {
+    return Proto.find_first_of("ajfrKLR") != std::string::npos;
+  }
----------------
"ajfrKLR" -> bingo ;-)

This probably makes sense, but who knows :-)
Not even sure if a comment makes things better here...


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D77594





More information about the cfe-commits mailing list