[compiler-rt] r245596 - [asan] Enable ASan for AArch64/42-bit VMA

Renato Golin via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 20 14:14:21 PDT 2015


On 20 August 2015 at 21:37, Renato Golin <renato.golin at linaro.org> wrote:
> On 20 August 2015 at 21:00, Pete Cooper via llvm-commits
> <llvm-commits at lists.llvm.org> wrote:
>> This is breaking our internal Darwin based bots.  I believe the reason is that this code has no else case, so we never set the variable.  There’s code elsewhere that checks for linux.  I wonder if there needs to be ‘elif not linux’ on the end here to support Darwin and other architectures?
>
> Oh, yes! Sorry about that.
>
> Adhemerval, do you have a fix handy?

Pete, should be fixed in r245614. Let me know.

cheers,
--renato


More information about the llvm-commits mailing list