[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 @@
+ break; //FIXME: is this correct? Can't a MachineBasicBlock have multiple return instructions?
> 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).
More information about the llvm-commits