D/ldc
1
0
Fork 0
mirror of https://github.com/xomboverlord/ldc.git synced 2026-01-12 02:43:14 +01:00
Code Issues Actions Packages Projects Releases Wiki Activity

Workaround for a bug in llvm 2.8 which appears only when all optimizations are off

Browse Source
This commit is contained in:
Alexey Prokhin
2010-11-02 13:20:21 +03:00
parent 6e8d25ee0d
commit e3afcf8a8d
1 changed files with 33 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

34
gen/toir.cpp
View File

@@ -40,6 +40,7 @@
#include "gen/nested.h"
#include "gen/utils.h"
#include "gen/warnings.h"
#include "gen/optimizer.h"
#include "llvm/Support/ManagedStatic.h"
@@ -1257,8 +1258,39 @@ DValue* IndexExp::toElem(IRState* p)
arrptr = DtoGEP(l->getRVal(), zero, r->getRVal());
}
else if (e1type->ty == Tarray) {
if(global.params.useArrayBounds)
if(global.params.useArrayBounds) {
#if 1
/*
Workaround for a bug in llvm 2.8 which appears only when
all optimizations are off.
Example:
int a[] = new int[5];
a[0] = 100;
a[1] = 1;
a[2] = 2;
a[3] = 3;
a[4] = 4;
for (int i = 0; i < a.length-1; i++)
printf("%d\n", a[i+1]); // Error would be around here
Output:
100
100
100
100
As you can see from the example, the index always is 0.
To avoid the issue we store the index before array bounds
checking.
*/
if (optLevel() == 0 && !r->isLVal()) {
LLValue *tmp = DtoAlloca(r->getType());
DtoStore(r->getRVal(), tmp);
r = new DVarValue(r->getType(), tmp);
}
#endif
DtoArrayBoundsCheck(loc, l, r, false);
}
arrptr = DtoArrayPtr(l);
arrptr = DtoGEP1(arrptr,r->getRVal());
}