[PATCH] D99008: [RISCV] [2/2] Add intrinsic for Zbr extension

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 25 16:44:17 PDT 2021


craig.topper added inline comments.


================
Comment at: clang/lib/Headers/riscv_zbr_intrin.h:9
+ */
+
+#ifndef __RISCV_ZBR_INTRIN_H
----------------
Please add 

```
#ifndef __RVINTRIN_H                                                                                                                                                                                                                                                              
#error "Never use <riscv_zbr_intrin.h> directly; include <rvintrin.h> instead."
#endif 
```

This file doesn't work if you include it by itself because it references int_xlen_t and __DEFAULT_FN_ATTRS which aren't defined in this file.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99008



More information about the llvm-commits mailing list