[PATCH] D23931: [XRay] ARM 32-bit no-Thumb support in LLVM

Dean Michael Berris via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 30 22:30:25 PDT 2016


dberris requested changes to this revision.

================
Comment at: lib/CodeGen/XRayInstrumentation.cpp:92
@@ +91,3 @@
+          TII->get(TargetOpcode::PATCHABLE_FUNCTION_EXIT));
+        break; //FIXME: is this correct? Can't a MachineBasicBlock have multiple return instructions?
+      }
----------------
dberris wrote:
> Yes, this is definitely not correct. This is a remnant of some refactoring I've done and it stuck around. :(
> 
> Let me add a test and fix, should be trivial.
This is now fixed in rL280192 -- please rebase to get the change (and tests).


https://reviews.llvm.org/D23931





More information about the llvm-commits mailing list