[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 19:18:38 PDT 2016


dberris added inline comments.

================
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?
+      }
----------------
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.


https://reviews.llvm.org/D23931





More information about the llvm-commits mailing list