[PATCH] D67661: [RISCV] Headers: Add Bitmanip extension Clang header files and rvintrin.h

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri added a comment.

Inline asm is //really// unfriendly to the optimizer.
Ideally the plan should be to incrementally getting rid of it as soon as backend learns to properly match particular builtin.



================
Comment at: clang-tools-extra/clang-include-fixer/find-all-symbols/STLPostfixHeaderMap.cpp:59-68
+      {"include/rv32bintrin-builtins.h$", "rv32bintrin-builtins.h"},
+      {"include/rv32bintrin-emulation.h$", "rv32bintrin-emulation.h"},
+      {"include/rv32bintrin.h$", "rv32bintrin.h"},
+      {"include/rv64bintrin-asm.h$", "rv64bintrin-asm.h"},
+      {"include/rv64bintrin-builtins.h$", "rv64bintrin-builtins.h"},
+      {"include/rv64bintrin-emulation.h$", "rv64bintrin-emulation.h"},
+      {"include/rv64bintrin.h$", "rv64bintrin.h"},
----------------
`<>` missing?


================
Comment at: clang-tools-extra/clangd/index/CanonicalIncludes.cpp:157-166
+          {"include/rv32bintrin-builtins.h", "rv32bintrin-builtins.h"},
+          {"include/rv32bintrin-emulation.h", "rv32bintrin-emulation.h"},
+          {"include/rv32bintrin.h", "rv32bintrin.h"},
+          {"include/rv64bintrin-asm.h", "rv64bintrin-asm.h"},
+          {"include/rv64bintrin-builtins.h", "rv64bintrin-builtins.h"},
+          {"include/rv64bintrin-emulation.h", "rv64bintrin-emulation.h"},
+          {"include/rv64bintrin.h", "rv64bintrin.h"},
----------------
`<>` missing?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D67661