This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH 0/X] Introduce HWASAN sanitizer to GCC


On 11/5/19 6:35 PM, Matthew Malcomson wrote:
On 05/11/2019 17:22, Martin Liška wrote:
On 11/5/19 5:11 PM, Matthew Malcomson wrote:
On 05/11/2019 15:10, Martin Liška wrote:
On 11/5/19 12:32 PM, Matthew Malcomson wrote:
Hello,

This patch series adds the LLVM hardware address sanitizer (HWASAN) to
GCC.  The document describing HWASAN can be found here
http://clang.llvm.org/docs/HardwareAssistedAddressSanitizerDesign.html.

Hello.

Thank you for the patch. I've just updated libsanitizer in GCC and I
also
prepared a patch which you have included in your series (but using an
older libsanitizer):
https://gcc.gnu.org/ml/gcc-patches/2019-11/msg00270.html.

Can you please rebase the series on top of the patch and exclude all the
cherry picks from
upstream? Plus you can now install
"[aarch64] Allocate enough space for err_str in
aarch64_handle_attr_branch_protection"
patch and exclude it from the series as well.
Will do!

I'm currently rebasing, but thought I should mention just in case you
didn't know that LLVM have fully moved over to git recently.

Hello.

You are right, I haven't noticed that.


There seem to be no commits on the LLVM svn repo since 2019-10-22, so
there are some cherry-picked hwasan patches that are not in at the point
you merged.

Do you miss any revisions that were installed in the new git repository
and are not in current GCC master? If so, I can update the merge.sh script
and do one more merge from trunk.

Martin

Yes, there are three commits in the new git repository and not in GCC
master that I need.  The most recent commit I need is 91167e2.

Good, I've just rewritten merge.sh to use the LLVM mono repository.
Tomorrow I'll test and install update from revision 82588e05cc32bb30807e480abd4e689b0dee132a.

Thanks,
Martin


Thanks,
Matthew.



Is this known, and we're taking the last SVN commit?

Cheers,
Matthew


Thanks,
Martin





Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]