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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to