- User Since
- Apr 25 2018, 1:47 PM (25 w, 1 d)
Tue, Oct 16
Mon, Oct 15
Sorry for the delay. Made patch https://reviews.llvm.org/D53299. Not
sure if I needed to make a patch for this or can commit right away,
but wanted to run it by you first.
I found those earlier when working on the monorepo. In fixing them now.
Fri, Oct 12
- Removed target hook
Thu, Oct 11
Wed, Oct 10
Tue, Oct 9
@ebevhan @rjmccall Seeing that the saturation intrinsic in https://reviews.llvm.org/D52286 should be put on hold for now, would it be fine to submit this patch as is? Then if the intrinsic is finished, come back to this and update this patch to use the intrinsic?
These are different per instance though. Even if merged, you're free to create one instance with options A and one instance with options B and run the first instance on a function pipeline and the second on a module pipeline.
Mon, Oct 8
I meant just merging the passes. In the sense that you have one pass class with one module run() method and one function run() method. The question then is whether you'll ever want to use different options for the different IRUnits.
Sun, Oct 7
Are there any other changes that would be necessary before getting revision acceptance?
Thu, Oct 4
This sounds like something that would be better handled in the build system rather than in source code. In particular, I think you don't actually want to detect "is this translation unit being compiled with ubsan enabled?", you instead want to detect "is the rest of musl libc being compiled with ubsan enabled?" -- you should not compile the stubs themselves with ubsan enabled, because ubsan might insert calls to its runtime at arbitrary places within that translation unit, which means you'll get infinite recursion when one of the ubsan callbacks ends up calling itself.
Wed, Oct 3
Tue, Oct 2
- Changed second argument of SSAT node from ConstantSDNode to VTSDNode
Mon, Oct 1
- Reduce size of patch by adding per-PM wrappers for AddressSanitizer and AddressSanitizerModule. The passes exposed to the new PM use opaque pointers to these wrappers.
- Added LLVM header description