[PATCH] D114154: [fir] Add fir.no_reassoc conversion

[Valentin Clement via Phabricator via llvm-commits]

clementval marked 3 inline comments as done.
clementval added inline comments.


================
Comment at: flang/test/Fir/convert-to-llvm.fir:1549
+// CHECK:         %[[ALLOC:.*]] = llvm.alloca %[[C1]] x i32 {in_type = i32, operand_segment_sizes = dense<0> : vector<2xi32>} : (i64) -> !llvm.ptr<i32>
+// CHECK-NOT:     fir.no_reassoc
+// CHECK:         %[[LOAD:.*]] = llvm.load %[[ARG0]] : !llvm.ptr<i32>
----------------
kiranchandramohan wrote:
> The reassoc is after the load right? Or are you simulating a reassociation here?
> 
> Anyway checking an fir op here does not help I think.
> 
Removed


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D114154