This revision was automatically updated to reflect the committed changes.
Closed by commit rG91167e22eca5: [hwasan] Remove lazy thread-initialisation
(authored by DavidSpickett).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D69574/new/
https://revie
pcc accepted this revision.
pcc added a comment.
This revision is now accepted and ready to land.
LGTM
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D69574/new/
https://reviews.llvm.org/D69574
___
cfe-c
mmalcomson added a comment.
Ping @pcc -- does this change to remove lazy thread initialisation look OK?
(I'm looking to start upstreaming hwasan instrumentation to GCC soon, and need
to know whether GCC must insert the thread initialisation code in function
prologues)
Repository:
rG LLVM Gi
eugenis added a comment.
LGTM keeping the flag looks like the right thing to do.
I'll leave this for @pcc to accept.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D69574/new/
https://reviews.llvm.org/D69574
___
mmalcomson created this revision.
mmalcomson added reviewers: eugenis, pcc, Sanitizers.
mmalcomson added a project: Sanitizers.
Herald added subscribers: llvm-commits, cfe-commits, jfb, hiraditya, srhines.
Herald added projects: clang, LLVM.
As I asked in the comments of https://reviews.llvm.org/D