Porting HWASan to Linux x86-64, the third of the three patches, clang part.
Details
Diff Detail
- Repository
- rC Clang
- Build Status
Buildable 16317 Build 16317: arc lint + arc unit
Event Timeline
test/Driver/asan.c | ||
---|---|---|
12 | I am not replacing them, I am making it consistent with all other sanitizers, which also support many more platforms than just x86_64-linux-gnu or *-unknown-linux, but have tests for that one platform only. |
test/Driver/asan.c | ||
---|---|---|
12 | I don't think this is a good argument for changing existing tests. It does not add any new coverage, and in this particular example it is not even consistent with surrounding RUN lines - those use i386 triple. It simply adds noise to code history. If you want to test that hwasan supports x86_64 triple - add a new test case. |
test/Driver/asan.c | ||
---|---|---|
12 | Ok, this file, I'm ambivalent about, it is not consistent already, but the other ones I do feel quite strongly about, they are become more consistent and consistency is good. |
Hi, I'm curious about why hwasan requires PIE. Is there any documents mentioned this? Thanks!
I did not realize hwasan was not mentioned in RequiresPIE before.
Could you add a test for it?