[Bug 287300] lang/ocaml: remove PIE_UNSAFE and fix minor style

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=287300

--- Comment #2 from Benjamin Jacobs <freebsd@dev.thsi.be> ---
Hi again,

So the PIE_UNSAFE does seem indeed not useful.
The =! construct had been suggested to me before the switch from bsd.ocaml.ml
to Mk/Uses, the former was hooked before the Mk/Uses machinery and hence I
believe this was a more cautious way (but not relevant anymore, indeed). The
good news is that I've finally been able to get access to a powerpc64 system,
and I've been able to confirm that everything works fine with clang only.
As mentioned, I had made a patch to fix the aarch64 assembler emitter (D48228)
so that it ocaml works with the base clang too. In other words, I don't think
it make sense to commit your patch as of now; I'd prefer to have  everyting
done (i.e. remove all the arch special-casing) as a single ocaml update.
I'm trying to find some time to prepare an new update to superseed D48228 which
would be more limited in scope (as that was the reason given to me to not act
on it) to move forward.

Best regards,

Benjamin

-- 
You are receiving this mail because:
You are the assignee for the bug.