Re: [PATCH v3 03/20] target/riscv: Introduce elp state and enabling controls for zicfilp

[Richard Henderson]

On 8/7/24 10:06, Deepak Gupta wrote:
> +/* enum for branch tracking state in cpu/hart */
> +typedef enum {
> +    NO_LP_EXPECTED = 0,
> +    LP_EXPECTED = 1,
> +} cfi_elp;

I know this is language is in the spec, but would it make more sense to use

  bool elp_expected;

?

If not, Coding Style requires CamelCase for typedefs.


r~