[PATCH] D50559: [gnu-objc] Make selector order deterministic.

2018-08-10 Thread Bernhard M. Wiedemann via Phabricator via cfe-commits
bmwiedemann added a comment. got a build failure with this patch added onto 6.0.1 Comment at: lib/CodeGen/CGObjCGNU.cpp:3547 + allSelectors.push_back(entry.first); +llvm::sort(allSelectors.begin(), allSelectors.end()); compilation failed here:

[PATCH] D50559: [gnu-objc] Make selector order deterministic.

2018-08-11 Thread Bernhard M. Wiedemann via Phabricator via cfe-commits
bmwiedemann added a comment. I got it compiled with std::sort on top of 6.0.1 and found that it indeed removes the indeterminism: When I compiled libobjc2-1.8.1/arc.m 25 times, I got the same md5sum every time - and the same result with and without ASLR. Repository: rC Clang

[PATCH] D52967: Extend shelf-life by 70 years

2018-11-06 Thread Bernhard M. Wiedemann via Phabricator via cfe-commits
bmwiedemann added a comment. I tested now, that it still works on i586 (in addition to x86_64) Repository: rC Clang https://reviews.llvm.org/D52967 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D52967: Extend shelf-life by 70 years

2018-10-06 Thread Bernhard M. Wiedemann via Phabricator via cfe-commits
bmwiedemann created this revision. bmwiedemann added a reviewer: RKSimon. Herald added a subscriber: cfe-commits. Without this patch, tests would fail after 2030. Note: requires timestamps with more than 31 bits Repository: rC Clang https://reviews.llvm.org/D52967 Files:

[PATCH] D52967: Extend shelf-life by 70 years

2018-12-01 Thread Bernhard M. Wiedemann via Phabricator via cfe-commits
bmwiedemann added a comment. As to the why: I'm working on reproducible builds for openSUSE and for that I verify that our packages can still give identical build results 15 years from now (which is the expected lifetime of today's enterprise software). `kvm -rtc base` option helps there.

[PATCH] D52967: Extend shelf-life by 70 years

2019-05-08 Thread Bernhard M. Wiedemann via Phabricator via cfe-commits
bmwiedemann accepted this revision. bmwiedemann added a comment. Herald added a subscriber: dexonsmith. Herald added a project: clang. Can someone please merge this change? Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D52967/new/ https://reviews.llvm.org/D52967