jyapayne/Nim
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Pick the and symbol we need explicitly (#8249)

Browse source 
Using getSysSym made the compiler pick a random `and` symbol: if the
symbol table is shuffled we may end up selecting one of the wrong
overloads.

Fixes #8246
This commit is contained in:
LemonBoy 2018-07-08 22:00:32 +02:00 • committed by Andreas Rumpf
commit 03c8fdc6cb
1 changed files with 2 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

4
compiler/cgmeth.nim
View file

@ -231,8 +231,8 @@ proc genDispatcher(g: ModuleGraph; methods: TSymSeq, relevantCols: IntSet): PSym
var paramLen = sonsLen(base.typ) var paramLen = sonsLen(base.typ)
var nilchecks = newNodeI(nkStmtList, base.info) var nilchecks = newNodeI(nkStmtList, base.info)
var disp = newNodeI(nkIfStmt, base.info) var disp = newNodeI(nkIfStmt, base.info)
var ands = getSysSym(g, unknownLineInfo(), "and") var ands = getSysMagic(g, unknownLineInfo(), "and", mAnd)
var iss = getSysSym(g, unknownLineInfo(), "of") var iss = getSysMagic(g, unknownLineInfo(), "of", mOf)
let boolType = getSysType(g, unknownLineInfo(), tyBool) let boolType = getSysType(g, unknownLineInfo(), tyBool)
for col in countup(1, paramLen - 1): for col in countup(1, paramLen - 1):
if contains(relevantCols, col): if contains(relevantCols, col):