[PATCH] D21723: [RFC] Enhance synchscope representation

Mehdi AMINI via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 15 12:52:16 PDT 2017


mehdi_amini added a comment.

Is there a MIR test?



================
Comment at: lib/CodeGen/MIRPrinter.cpp:709
+    const LLVMContext &Context =
+        MI.getParent()->getParent()->getFunction()->getContext();
     bool NeedComma = false;
----------------
I suspect Using `MF` would be shorter here.


================
Comment at: test/Bitcode/atomic-no-syncscope.ll:1
+; RUN: llvm-dis -o - %s.bc | FileCheck %s
+
----------------
Can you add a one line overview of what this test is doing?


https://reviews.llvm.org/D21723





More information about the llvm-commits mailing list