In order to emit overflow warnings from assignment statements whose
right-hand sides are constants that undergo conversions, run the
right-hand sides of assignments through constant folding after the
conversions have been made explicit in expression analysis.
Details
- Reviewers
jeanPerier - Commits
- rGb90ebbc5bd6d: [flang] Fold the RHS of assignment
Diff Detail
Event Timeline
flang/test/Lower/array-temp.f90 | ||
---|---|---|
121–123 | Regression test are failing. This is because the test update is now testing for something that is environment dependent: the path name of the current file. You will need to remove the checks from: ! CHECK: %[[V_47:[0-9]+]] = fir.address_of ... to %[[V_49:[0-9]+]] ... and replace that last one by: You will need to do the same for all _FortranAioBeginExternalListOutput calls (in general, this needs to be done on every runtime call that takes the file name as argument). This test is in general checking for way too much... sorry about the trouble this is causing you. New lowering test should be more focus on what they are testing. |
Regression test are failing. This is because the test update is now testing for something that is environment dependent: the path name of the current file.
You will need to remove the checks from: ! CHECK: %[[V_47:[0-9]+]] = fir.address_of ... to %[[V_49:[0-9]+]] ... and replace that last one by:
%[[V_49:[0-9]+]] = fir.call @_FortranAioBeginExternalListOutput(%[[C_m1_i32]]], %{{.*}}, %{{.*}}) {{.*}}: (i32, !fir.ref<i8>, i32) -> !fir.ref<i8>
You will need to do the same for all _FortranAioBeginExternalListOutput calls (in general, this needs to be done on every runtime call that takes the file name as argument).
This test is in general checking for way too much... sorry about the trouble this is causing you. New lowering test should be more focus on what they are testing.