@Oli I don't get your method proposal
are you saying it should be a method returning
If it is a method to call on the list fields before iteration, I think I'd rather remove this entirely and do sorting on the miri side.
that's probably more elegant anyway...
hm, except that it would be even nicer to use https://doc.rust-lang.org/nightly/nightly-rustc/rustc_target/abi/enum.FieldPlacement.html#method.index_by_increasing_offset
I do dislike having the code in the miri-engine. I meant a method that does the sorting, yes.
I mean you need a
Vec at the end anyway, right? So there's no problem with impl trait or sth
the code in miri can use
index_by_increasing_offset, and miri-engine can just do the regular thing
do we expect the miri-engine to sort some day in the future, too?
if so, just unconditionally sort with the comment as an explanation
I'll just revert that commit and do it all in miri without a new hook, seems simpler
index_by_increasing_offset does not worki because I cannot use impl Trait in a trait