[PATCH] D26805: [XRay][AArch64] Implemented a test for the compile-time sleds emitted, and fixed a bug in the jump instruction

Serge Rogatch via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 18 09:02:20 PST 2016


rSerge added inline comments.


================
Comment at: lib/Target/AArch64/AArch64AsmPrinter.cpp:228
+  // including the current instruction.
+  EmitToStreamer(*OutStreamer, MCInstBuilder(AArch64::B).addImm(32 >> 2));
 
----------------
dberris wrote:
> Why not just write out the actual number of instructions? i.e. why not just: `.addImm(8)` ?
Ok, changing. I just thought that it's more convenient to think in bytes and show that that number is shifted by 2.


================
Comment at: test/CodeGen/AArch64/xray-attribute-instrumentation.ll:26
+; CHECK-NEXT:  ret
+}
----------------
dberris wrote:
> You might want to make sure that the xray_instr_map section is written out and that you're seeing the references to the correct labels?
Changing.


https://reviews.llvm.org/D26805





More information about the llvm-commits mailing list