6502/llvm-6502
1
0
Fork 0
mirror of https://github.com/c64scene-ar/llvm-6502.git synced 2025-05-13 17:38:39 +00:00
Code Issues Projects Releases Wiki Activity

A FIXME about block addresses and indirectbr.

Browse Source 
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@142569 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Eli Friedman 2011-10-20 04:05:33 +00:00
parent 400ea5bde0
commit 531afb166c
1 changed files with 6 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
lib/Analysis/InlineCost.cpp
View File

@ -135,6 +135,12 @@ void CodeMetrics::analyzeBasicBlock(const BasicBlock *BB,
// for example) would be referring to the original function, and this indirect // for example) would be referring to the original function, and this indirect
// jump would jump from the inlined copy of the function into the original // jump would jump from the inlined copy of the function into the original
// function which is extremely undefined behavior. // function which is extremely undefined behavior.
// FIXME: This logic isn't really right; we can safely inline functions
// with indirectbr's as long as no other function or global references the
// blockaddress of a block within the current function. And as a QOI issue,
// if someone is using a blockaddress wihtout an indirectbr, and that
// reference somehow ends up in another function or global, we probably
// don't want to inline this function.
if (isa<IndirectBrInst>(BB->getTerminator())) if (isa<IndirectBrInst>(BB->getTerminator()))
containsIndirectBr = true; containsIndirectBr = true;