[PATCH] D108083: add sanitizer support to hexagon

[Brian Cain via Phabricator via cfe-commits]

bcain added a comment.

In D108083#2947907 <https://reviews.llvm.org/D108083#2947907>, @vitalybuka wrote:

> Can you cut this into separate peaces, at least patch per sanitizer for "asan lsan ubsan scudo scudo_standalone cfi safestack"?

Sure, that's no problem. While I'm breaking it up, us there a guideline/preference to whether the clang changes can be combined in one of these patches, or should those be on their own as well?

> Some common peaces could be probably extracted as well.





================
Comment at: compiler-rt/test/asan/CMakeLists.txt:19
     set(${bits} 64)
-  elseif (${arch} MATCHES "i386|arm|mips|mipsel|sparc")
+  elseif (${arch} MATCHES "i386|arm|mips|mipsel|sparc|hexagon")
     set(${bits} 32)
----------------
vitalybuka wrote:
> why only this test changes?
> 
Sorry, I shouldnt have included this, I'll omit it. I'd like to land the build changes before test ones. 


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D108083/new/

https://reviews.llvm.org/D108083