Antoni Boucher (antoyo)) commented on the code:

> +++ gcc/config/aarch64/aarch64-jit.cc
> @@ -0,0 +60,4 @@
> +  FEATURE_STRING);
> +#include "aarch64-option-extensions.def"
> +
> +  if (TARGET_BTI)
@pinskia: `TARGET_BTI` is defined in `aarch64.h`, so I assume it is OK to 
define it manually here.
Unless I'm missing something?

> +++ gcc/config/aarch64/aarch64-jit.cc
> @@ -0,0 +60,4 @@
> +  FEATURE_STRING);
> +#include "aarch64-option-extensions.def"
> +
> +  if (TARGET_BTI)
Yes. BTI should be fine as it is only enabled if armv8.5-a is enabled.


--
https://forge.sourceware.org/gcc/gcc-TEST/pulls/108#issuecomment-3325

Reply via email to