)]}'
{"build/toolchain/llvm-efi/cc_toolchain_config.bzl":[{"author":{"_account_id":1000001,"name":"Lorenz Brun","display_name":"Lorenz","email":"lorenz@monogon.tech","username":"lorenz","avatars":[{"url":"https://www.gravatar.com/avatar/75c04f6e9881c24ee621fba80667eed8.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/75c04f6e9881c24ee621fba80667eed8.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/75c04f6e9881c24ee621fba80667eed8.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/75c04f6e9881c24ee621fba80667eed8.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"change_message_id":"5948bcacfee9f498e2e28d513c8c9389d38512b3","unresolved":true,"context_lines":[{"line_number":131,"context_line":"                            # them is used for intermediary files, and one for"},{"line_number":132,"context_line":"                            # actual PE/COFF files?"},{"line_number":133,"context_line":"                            \"-Wl,/Brepro\","},{"line_number":134,"context_line":"                            \"-Wl,--no-insert-timestamp\","},{"line_number":135,"context_line":"                            \"-nostdlib\","},{"line_number":136,"context_line":"                            \"build/toolchain/llvm-efi/fltused.o\","},{"line_number":137,"context_line":"                        ],"}],"source_content_type":"text/x-python","patch_set":1,"id":"600b3548_f9e10f21","line":134,"range":{"start_line":134,"start_character":33,"end_line":134,"end_character":54},"updated":"2023-12-19 16:48:36.000000000","message":"Looking at the LLVM source code (especially https://github.com/llvm/llvm-project/commit/c8dd6ccc8a358273114ef7e6f19ad59f6258048c) `/Brepro` inserts a hash (xxhash) of the file as the \"timestamp\". So `--no-insert-timestamp` which results in `/timestamp:0` being appended by the driver is only necessary if we want to explicitly zero that value. In my quick test it was reproducible without it. It also seems to resolve a warning from lld-link. So I would recommend dropping that, but I\u0027ll leave the final decision up to you.","commit_id":"59a4cff3f4ddb849c7f2a83f3419c17ce75c4d99"},{"author":{"_account_id":1000002,"name":"Serge Bazanski","display_name":"Serge","email":"serge@monogon.tech","username":"serge","avatars":[{"url":"https://www.gravatar.com/avatar/52c41428b6369f2c02b9717425216f7d.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/52c41428b6369f2c02b9717425216f7d.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/52c41428b6369f2c02b9717425216f7d.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/52c41428b6369f2c02b9717425216f7d.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"change_message_id":"dd4221f98643a3d83739b8894b3c39755a315bfe","unresolved":false,"context_lines":[{"line_number":131,"context_line":"                            # them is used for intermediary files, and one for"},{"line_number":132,"context_line":"                            # actual PE/COFF files?"},{"line_number":133,"context_line":"                            \"-Wl,/Brepro\","},{"line_number":134,"context_line":"                            \"-Wl,--no-insert-timestamp\","},{"line_number":135,"context_line":"                            \"-nostdlib\","},{"line_number":136,"context_line":"                            \"build/toolchain/llvm-efi/fltused.o\","},{"line_number":137,"context_line":"                        ],"}],"source_content_type":"text/x-python","patch_set":1,"id":"24967621_bb85750c","line":134,"range":{"start_line":134,"start_character":33,"end_line":134,"end_character":54},"in_reply_to":"600b3548_f9e10f21","updated":"2023-12-21 13:17:25.000000000","message":"Done","commit_id":"59a4cff3f4ddb849c7f2a83f3419c17ce75c4d99"}]}
