Bryan Chan

Results 25 issues of Bryan Chan

Do not reorder non-constant operands, even for commutative opcodes. Doing so breaks short-circuit evaluation of logical expressions, and can also cause side effects to occur out of order. This PR...

Line 45 in `test/f90_correct/src/dt17.f90` contains a rank mismatch which causes Flang to emit a warning: ``` F90-W-0155-The number of subscripts is less than the rank of ia (test/f90_correct/src/dt17.f90: 45) 0...

Some general observation regarding the use of the bool type. Some of the files try to do 'the right thing', by adding inclusion of the stdbool.h header and even handle...

Setting the CMake variable `WITH_WERROR=ON` tells Clang to compile Flang code with `-Werror`, which turns compile-time warnings into fatal errors. This had to be turned off by default in b8b491466da3...

From https://github.com/flang-compiler/flang/pull/1127#issuecomment-914347599: > [Flang is] missing support for UNC paths like \\machine\mountpoint\file, which to my best knowledge are mainly used when files on a non-local drive are to be referenced...

windows

Classic Flang contains two implementations of hashset and hashmap that are almost identical: - https://github.com/flang-compiler/flang/blob/master/tools/flang1/flang1exe/hash.c - https://github.com/flang-compiler/flang/blob/master/lib/ADT/hash.c The first one is used by flang1, and the second is used by...

There are several issues with the `-Mbackslash` option. 1. Currently, Flang's default behaviour is not compatible with that of gfortran and ifort (as explained in [Arm's own documentation](https://developer.arm.com/documentation/101726/0210/Port-and-Optimize-your-Application-to-SVE-enabled-Arm-based-processors/Fortran-compiler-migration-guides/armflang-for-ifort-users#urj1591008865151__handling-backslash-characters)). Some of...

This test program crashes with a segmentation fault when compiled with the latest Flang on X86: ```fortran program example implicit none character(:), pointer :: ptr(:) character(3), target :: array(1) =...

```fortran module binops contains integer function add(i, j) integer :: i, j add = i + j end function integer function multiply(i, j) integer :: i, j multiply = i...

Section 8.4 ("Initialization") of the Fortran 2018 standard states: > If `null-init` appears, the initial association status of the object is disassociated. If `initial-data-target` appears, the object is initially associated...