Hey Michael, I'd bet this is from C for loops. I added new codegen functionality for c for loops in stmt.cpp and I didn't think about llvm (sorry about that.) See stmt.cpp: BlockStmt::codegen and codegenCForLoopHeaderSegment
I don't think we can add a minimal modules test as ranges aren't in the minimal modules. I also can't think of a clean or simple way to disable c for loops (we would have to special case code in the modules, and a large routine in lowerIterators.cpp) I'm building llvm now and I'll take a look at what all would have to change to get llvm working again. I'm pretty sure we just need to add a case for c for loops in BlockStmt::codegen. I'll let you know what I see once llvm is built and we can work off list to resolve this. Elliot -----Original Message----- From: Michael Ferguson <[email protected]> Date: Wednesday, August 20, 2014 at 8:21 AM To: Elliot Ronaghan <[email protected]> Cc: chapel-developers <[email protected]> Subject: LLVM compilation >Hi Elliot - > >I suspect that CForLoops-V1 has broken --llvm compilation. >Do you know that to be the case - in other words are there >places you added something for the C backend only? It's possible >that the problem is another recent change, such as Tom's >constructor updates... > >Right now I get: >$ chpl --llvm hello.chpl >$ ./a.out >.../localeModels/flat/LocaleModel.chpl:198: error: halt reached - array >index out of bounds: (13956352) > >It would help a lot to have a --minimal-modules test >of the new C for loop functionality... or a way to temporarily disable the >emission of C for loops except for in a specific test. > >Any pointers or test cases would be appreciated. > >Thanks, > >-michael ------------------------------------------------------------------------------ Slashdot TV. Video for Nerds. Stuff that matters. http://tv.slashdot.org/ _______________________________________________ Chapel-developers mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/chapel-developers
