This revision was automatically updated to reflect the committed changes.
Closed by commit rGb06fbdeae752: [compiler-rt][hwasan] Add InitState options to
thread initialization (authored by leonardchan).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
eugenis accepted this revision.
eugenis 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/D104553/new/
https://reviews.llvm.org/D104553
leonardchan added a comment.
*ping*
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D104553/new/
https://reviews.llvm.org/D104553
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
leonardchan updated this revision to Diff 353421.
leonardchan marked an inline comment as done.
leonardchan added a comment.
Updated and rebased.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D104553/new/
https://reviews.llvm.org/D104553
Files:
phosek added inline comments.
Comment at: compiler-rt/lib/hwasan/hwasan_linux.cpp:430
-void Thread::InitStackAndTls() {
+void Thread::InitStackAndTls(const InitState *state) {
uptr tls_size;
If it's unused then you can omit the argument name to avoid the
leonardchan created this revision.
leonardchan added reviewers: phosek, mcgrathr, vitalybuka, eugenis.
leonardchan added a project: Sanitizers.
Herald added a subscriber: dberris.
leonardchan requested review of this revision.
Herald added a subscriber: Sanitizers.
Similar to `InitOptions` in