search

discopop-project / discopop

DiscoPoP - Discovery of Potential Parallelism
http://www.discopop.org
BSD 3-Clause "New" or "Revised" License
40 stars 22 forks source link

about CFG #666

Closed RainMolisa closed 4 weeks ago

RainMolisa commented 1 month ago

Hello Discopop! I have a problem about InstructionCFG.cpp. I find llvm have CFG.h too, so why you don't use llvm's CFG.h ? What's the defference about InstructionCFG.cpp and CFG.h. (/llvm/include/llvm/Analysis/CFG.h)

Additional, I find the function 'void InstructionCFG::findAndAddFirstRelevantInstructionInSuccessorBlocks' have a problem that is it run into unlimited recursion in some case. I'm sorry I can't make a case, because the target code is big. I will extract the problem case in future.

best wish!

RainMolisa commented 4 weeks ago

think you for you new code, 'void InstructionCFG::findAndAddFirstRelevantInstructionInSuccessorBlocks' unlimited recursion problem has been fixed