[PATCH] D137316: [Clang][LoongArch] Implement __builtin_loongarch_crc_w_d_w builtin and add diagnostics

Gong LingQin via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 7 04:16:54 PST 2022


gonglingqin added inline comments.


================
Comment at: clang/lib/Basic/Targets/LoongArch.cpp:174
+    const std::vector<std::string> &FeaturesVec) const {
+
+  if (getTriple().getArch() == llvm::Triple::loongarch64)
----------------
SixWeining wrote:
> Delete blank line.
Thanks, I will delete it.


================
Comment at: clang/lib/Sema/SemaChecking.cpp:3677
+                                             CallExpr *TheCall) {
+
+  switch (BuiltinID) {
----------------
SixWeining wrote:
> Delete blank line.
Thanks, I will delete it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D137316



More information about the cfe-commits mailing list