[PATCH] D48383: [Dominators] Add the DomTreeUpdater class

Vedant Kumar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 20 14:44:19 PDT 2018


vsk added inline comments.


================
Comment at: unittests/IR/DomTreeUpdaterTest.cpp:608
+  StringRef FuncName = "f";
+  StringRef ModuleString = "define i32 @f(i32 %i, i32 *%p) {\n"
+                           " bb0:\n"
----------------
Just a minor stylistic point, but it can be neater to use the extended literal syntax for IR snippets, like this:
```
foo = R"(
define i32 @f(i32 %i, i32 *%p) {
bb0:
    store i32 %i, i32 *%p
...
)";
```


Repository:
  rL LLVM

https://reviews.llvm.org/D48383





More information about the llvm-commits mailing list