Adding 40-bit shadow memory parameters because MIPS64 uses 40-bit virtual memory addresses.
Diff Detail
Diff Detail
- Repository
- rL LLVM
Event Timeline
lib/Transforms/Instrumentation/EfficiencySanitizer.cpp | ||
---|---|---|
102 | Style: { on prior line |
Comment Actions
Removed default case for reporting error for unsupported architectures as this was causing failures on aarch64, powerpc64 build bots
Comment Actions
Theres no good reason for restricting architectures other than x86 and mips. Since the instrumentation tests are running cleanly without these shadow memory parameters, I am okay with removing the default case which is reporting the fatal error.
Comment Actions
Using if ... else construct instead of switch case to avoid warning about unhandled enumeration values.
Comment Actions
Sorry, I was out for more than a week. The patch looks ok now, assuming it still passes all tests on X86_64, AArch64 and MIPS64.
Style: { on prior line