On 3/10/25 08:17, Richard Henderson wrote:
On 3/9/25 21:58, Pierrick Bouvier wrote:
They are now accessible through exec/memory.h instead, and we make sure
all variants are available for common or target dependent code.
...
diff --git a/include/exec/memory_ldst.h.inc b/include/exec/memory_ldst.h.inc
index 92ad74e9560..74519a88de0 100644
--- a/include/exec/memory_ldst.h.inc
+++ b/include/exec/memory_ldst.h.inc
@@ -19,7 +19,8 @@
* License along with this library; if not, see
<http://www.gnu.org/licenses/>.
*/
-#ifdef TARGET_ENDIANNESS
+uint8_t glue(address_space_ldub, SUFFIX)(ARG1_DECL,
+ hwaddr addr, MemTxAttrs attrs, MemTxResult *result);
uint16_t glue(address_space_lduw, SUFFIX)(ARG1_DECL,
hwaddr addr, MemTxAttrs attrs, MemTxResult *result);
You shouldn't be exposing
address_space_lduw
to common code, only
address_space_lduw_be
address_space_lduw_le
etc. I'm not sure what you're trying to do here.
r~
As mentioned in the cover letter, the goal is to extract memory
functions from cpu headers. The result is that we need to expose all
functions for both common and target dependent code.
In case you don't see the point, you can try to remove memory_ldst
include from cpu-all.h (without touching memory_ldst.h) and see the
compilation errors. This patch is the minimal change to get something
working.
If your point is that non be/le variants should be eliminated
completely, yes, it can be done after this series.
