[PATCH] D123544: [randstruct] Automatically randomize a structure of function pointers
Fangrui Song via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Apr 12 11:17:18 PDT 2022
MaskRay added a comment.
In D123544#3446285 <https://reviews.llvm.org/D123544#3446285>, @lebedev.ri wrote:
> Does this not lead to non-deterministic/non-reproducible builds?
> I do not understand why this feature must be inflicted onto everyone.
It seems that the feature fixes the randomization seed in the option `-frandomize-layout-seed=`, so it is deterministic, but it currently has a possible Windows vs non-Windows host difference, probably related to MSVC's `std::mt19937`.
My concern is related to the lack of `-frandomize-layout-seed-file=` testing in the base patch.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D123544/new/
https://reviews.llvm.org/D123544
More information about the cfe-commits
mailing list