[flang-commits] [PATCH] D117159: [flang] Extension: initialization of LOGICAL with INTEGER & vice versa

[Jean Perier via Phabricator via flang-commits]

jeanPerier accepted this revision.
jeanPerier added a comment.
This revision is now accepted and ready to land.

Please check why the buildbots are failing in `data-to-inits.cpp`. Logic looks good to me.
We will have to do something in lowering to support this feature, I think lowering would currently canonicalize the value when lowering initializers.



================
Comment at: flang/lib/Semantics/data-to-inits.cpp:242
+std::optional<std::pair<SomeExpr, bool /* converted character */>>
+DataInitializationCompiler<DSV>::ConvertElement(
     const SomeExpr &expr, const evaluate::DynamicType &type) {
----------------
LLVM builds raise error `expected ';' at end of declaration` here (see bots).


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

https://reviews.llvm.org/D117159