[PATCH] D66084: Remove asserts in getLoopGuardBranch

Whitney Tsang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 19 06:35:43 PDT 2019


Whitney added inline comments.


================
Comment at: unittests/Analysis/LoopInfoTest.cpp:1157
+  const char *ModuleStr =
+      "target datalayout = \"e-m:o-i64:64-f80:128-n8:16:32:64-S128\"\n"
+      "define void @foo(i32 %n) {\n"
----------------
DTharun wrote:
> Whitney wrote:
> > is this needed?
> Do you mean the data layout?
That's right, I meant the data layout.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D66084/new/

https://reviews.llvm.org/D66084





More information about the llvm-commits mailing list