diff options
| -rw-r--r-- | lib/CodeGen/BranchFolding.cpp | 2 | ||||
| -rw-r--r-- | lib/CodeGen/README.txt | 8 |
2 files changed, 9 insertions, 1 deletions
diff --git a/lib/CodeGen/BranchFolding.cpp b/lib/CodeGen/BranchFolding.cpp index 021001d15f..d4b1e90fd5 100644 --- a/lib/CodeGen/BranchFolding.cpp +++ b/lib/CodeGen/BranchFolding.cpp @@ -272,7 +272,7 @@ static unsigned ComputeCommonTailLength(MachineBasicBlock *MBB1, while (I1 != MBB1->begin() && I2 != MBB2->begin()) { --I1; --I2; if (!I1->isIdenticalTo(I2) || - // XXX: This check is dubious. It's used to get around a problem where + // FIXME: This check is dubious. It's used to get around a problem where // people incorrectly expect inline asm directives to remain in the same // relative order. This is untenable because normal compiler // optimizations (like this one) may reorder and/or merge these diff --git a/lib/CodeGen/README.txt b/lib/CodeGen/README.txt index 1e2837580a..fc59cdf864 100644 --- a/lib/CodeGen/README.txt +++ b/lib/CodeGen/README.txt @@ -179,3 +179,11 @@ work. The ocaml frametable structure supports liveness information. It would be good to support it. + +//===---------------------------------------------------------------------===// + +The FIXME in ComputeCommonTailLength in BranchFolding.cpp needs to be +revisited. The check is there to work around a misuse of directives in inline +assembly. + +//===---------------------------------------------------------------------===// |