This only covers the common load/store case. There will be further patches required for masked load/store and some of the fast-path optimization cases.
Details
Details
Diff Detail
Diff Detail
- Repository
 - rG LLVM Github Monorepo
 
Event Timeline
| llvm/lib/Transforms/Instrumentation/AddressSanitizer.cpp | ||
|---|---|---|
| 1425 | Should we call TypeStoreSize.getFixedValue() while we're here instead of using implicit conversion to unsigned?  | |
| llvm/lib/Transforms/Instrumentation/AddressSanitizer.cpp | ||
|---|---|---|
| 1425 | Honestly, it should probably be a switch inside a condition. Mind if I do that as a separate post commit? I'd like to keep the change isolated here.  | |
| llvm/lib/Transforms/Instrumentation/AddressSanitizer.cpp | ||
|---|---|---|
| 1425 | Sure.  | |
Comment Actions
Changes are look reasonable and good to me, but I am also new for ASan too, I guess we need few more eye to review.
Should we call TypeStoreSize.getFixedValue() while we're here instead of using implicit conversion to unsigned?