Add the rest of intrinsics to clang except intrinsics using vector mask
registers.
Details
- Reviewers
simoll efocht - Commits
- rGc2f62ab84b5f: [Clang][VE] Add the rest of intrinsics to clang
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
clang/include/clang/Basic/BuiltinsVEVL.gen.def | ||
---|---|---|
1242 | I don't want to add at the top of this file bacause this file is automatically generated and I don't want to use any structured comments. But, I added comments at the each point of #if 0 and added comments at the place where include this file. | |
clang/lib/Headers/velintrin.h | ||
65 | I guess no. So, I changed the code following your suggestion. |
clang/include/clang/Basic/BuiltinsVEVL.gen.def | ||
---|---|---|
35 | Why leave dead code under comments? Bad habit, remove it please. |
clang/include/clang/Basic/BuiltinsVEVL.gen.def | ||
---|---|---|
35 | Thank you for the comments. What is the best way to leave a list of functions need to be supported in near future? Should I just make a list of function names using vector mask registers instead of writing them down with "BUILTIN" for future reference? |
@kaz7 With rG0aab34410403 in tree you should be able to enable the masked intrinsics now.
Why leave dead code under comments? Bad habit, remove it please.